Add addModulesDirectories config

[neezer]

Allows appending to the default list of modules directories. I picked a name that I think sounds good, but happy to change it if y'all have something else you'd prefer. 😄

Relevant discussion: #195

@RyanZim

[michael-ciniawsky]

Maybe we could refactor options.resolve for that , instead of adding it to options directly?

let resolveOpt = {} // defaults

if (typeof options.resolve === 'object') {
   resolveOpt = Object.assign(resolveOpt, options.resolve)
   resolveOpt.moduleDirectories.push('node_modules', 'web_modules')
}

if (typeof options.resolve === 'function') {
  // Use custom resolver
}

[neezer]

Neat, @michael-ciniawsky! Happy to change the PR to this approach.

@RyanZim @MoOx Any preference?

[RyanZim]

@michael-ciniawsky I like that idea, I just think using the resolve option is rather implicit behavior.

If we go that direction, we should set root, and path in the same object as moduleDirectories.

For right now, I would be in favor of just doing it the way it is done in this PR. For the next major version, I would consider removing root, path, and addModuleDirectories, in favor of a single resolveOpts option; bikeshed about that later.

[RyanZim]

LGTM, except for the comment below.

[RyanZim]

Probably should link to https://github.com/substack/node-resolve instead, since that is the module that is actually used.

[neezer]

@RyanZim Updated!

[RyanZim]

@ai Thoughts? @MoOx Any objections?

[RyanZim]

@neezer Thanks!

[renatorib]

It's not published yet?

[RyanZim]

@renatorib It's not. I should publish it. I was planning on adding a few other things to the next release, but I haven't got around to it, so I guess I'll just publish this. I'll comment here as soon as I get it done.

[RyanZim]

@renatorib @neezer Released: https://github.com/postcss/postcss-import/releases/tag/9.1.0. Sorry for the long wait.

[neezer]

@RyanZim Great! So should #195 be closed now?

[RyanZim]

@neezer Yep, thanks.