Implements the pnp hook for the resolve package
#6816
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
|
In the meantime until the {
"resolutions": {
"resolve": "npm:resolve@1.9.0"
}
} |
|
Just FYI: this also fixes an issue with specifying plugin names in a babelrc file (in my case it was .babelrc.js). The issue was due to the same resolve package being used here: https://github.com/babel/babel/blob/e85faec47d5d3ef940b7a85d48fa24e6e1cc32ab/packages/babel-core/src/config/files/plugins.js#L99 After this fix it works great and you do not have to specify require.resolve for your plugins/presets. |
|
resolve v1.9 was released last night, fwiw |
asfgit
pushed a commit
to apache/camel-website
that referenced
this pull request
Jan 4, 2019
Yarn 13+ patches `resolve` module to resolve from PnP[1]. [1] yarnpkg/yarn#6816
asfgit
pushed a commit
to apache/camel-website
that referenced
this pull request
Jan 4, 2019
Yarn 13+ patches `resolve` module to resolve from PnP[1]. [1] yarnpkg/yarn#6816
asfgit
pushed a commit
to apache/camel-website
that referenced
this pull request
Jan 4, 2019
Yarn 13+ patches `resolve` module to resolve from PnP[1]. [1] yarnpkg/yarn#6816
asfgit
pushed a commit
to apache/camel-website
that referenced
this pull request
Jan 4, 2019
Yarn 13+ patches `resolve` module to resolve from PnP[1]. [1] yarnpkg/yarn#6816
asfgit
pushed a commit
to apache/camel-website
that referenced
this pull request
Jan 7, 2019
Yarn 13+ patches `resolve` module to resolve from PnP[1]. [1] yarnpkg/yarn#6816
netbsd-srcmastr
pushed a commit
to NetBSD/pkgsrc
that referenced
this pull request
Jan 17, 2019
## 1.13.0 - Implements a new `package.json` field: `peerDependenciesMeta` [6671](yarnpkg/yarn#6671) - [**Maël Nison**](https://twitter.com/arcanis) - Adds an `optional` settings to `peerDependenciesMeta` to silence missing peer dependency warnings [6671](yarnpkg/yarn#6671) - [**Maël Nison**](https://twitter.com/arcanis) - Implements `yarn policies set-version [range]`. Check [the documentation]() for usage & tips. [6673](yarnpkg/yarn#6673) - [**Maël Nison**](https://twitter.com/arcanis) - Fixes a resolution issue when a package had an invalid `main` entry [6682](yarnpkg/yarn#6682) - [**Maël Nison**](https://twitter.com/arcanis) - Decreases the size of the generated `$PATH` environment variable for a better Windows support [6683](yarnpkg/yarn#6683) - [**Rowan Lonsdale**](https://github.com/hWorblehat) - Fixes postinstall scripts for third-party packages when they were referencing a binary from their own dependencies [6712](yarnpkg/yarn#6712) - [**Maël Nison**](https://twitter.com/arcanis) - Fixes yarn audit exit code overflow [6748](yarnpkg/yarn#6748) - [**Andrey Vetlugin**](https://github.com/antrew) - Stops automatically unplugging packages with postinstall script when running under `--ignore-scripts` [6820](yarnpkg/yarn#6820) - [**Maël Nison**](https://twitter.com/arcanis) - Adds transparent support for the [`resolve`](https://github.com/browserify/resolve) package when using Plug'n'Play [6816](yarnpkg/yarn#6816) - [**Maël Nison**](https://twitter.com/arcanis) - Properly reports the error codes when the npm registry throws 500's [6817](yarnpkg/yarn#6817) - [**Maël Nison**](https://twitter.com/arcanis)
# for free
to join this conversation on GitHub.
Already have an account?
# to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds builtin
resolvesupport using the mechanisms described in browserify/resolve#174. I've also added an escape hatch (forceNodeResolution) in case something goes awry.cc @ljharb
Test plan
Added an automated test.