Release/v7.17.0 #3398
Merged
Release/v7.17.0 #3398
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
…es in the repo PR-URL: #3364 Credit: @nlf Close: #3364 Reviewed-by: @wraithgar
There is code in all of these places we control so it should be linted PR-URL: #3367 Credit: @wraithgar Close: #3367 Reviewed-by: @isaacs
We bundle our deps, but we don't need to bundle docs, changelogs, editorconfigs, test coverage reports, .github workflow definitions, lint configurations, and all the rest, which we never use. This cuts about 10% off of our publish artifact file size. ``` $ ls -laF npm-7.16.0-*.tgz -rw-r--r-- 1 isaacs staff 7174497 Jun 3 13:01 npm-7.16.0-release-next.tgz -rw-r--r-- 1 isaacs staff 6782377 Jun 3 13:00 npm-7.16.0-trim-node-modules.tgz $ ls -laF npm-7.16.0-*.tar -rw-r--r-- 1 isaacs staff 19020288 Jun 3 13:01 npm-7.16.0-release-next.tar -rw-r--r-- 1 isaacs staff 17474048 Jun 3 13:00 npm-7.16.0-trim-node-modules.tar ``` PR-URL: #3362 Credit: @isaacs Close: #3362 Reviewed-by: @nlf
We have a fairly small number of things that we wish to ship in our package, and they're all nicely arranged in a small number of folders. Instead of having a sprawling .npmignore file, we should just get with the recommended best practice, and use a "files" list in package.json. PR-URL: #3365 Credit: @isaacs Close: #3365 Reviewed-by: @wraithgar
setCredentialsByURI was clobbering the saving of the scope:registry config PR-URL: #3383 Credit: @wraithgar Close: #3383 Reviewed-by: @nlf
Refactored a bit so that we can more easily change the `top` and `prefix` params that were being used, and are different under the workspace context. PR-URL: #3368 Credit: @wraithgar Close: #3368 Reviewed-by: @ruyadorno
Added examples of using `npm init` to bootstrap a new workspace and a section on how to add/manage dependencies to workspaces. PR-URL: #3392 Credit: @ruyadorno Close: #3392 Reviewed-by: @wraithgar
ruyadorno
approved these changes
Jun 10, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in 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.
v7.17.0 (2021-06-10)
FEATURES
ef668ab57#3368 feat(diff): add workspace support (@wraithgar)BUG FIXES
26d00c477#3364 fix(tests): mock writeFile in pack tests so we dont create 0 byte files in the repo (@nlf)f130a81d6#3367 fix(linting): add scripts, docs, smoke-tests (@wraithgar)992799cd8#3383 fix(login): properly save scope if defined (@wraithgar)DOCUMENTATION
844229519#3392 docs(workspaces): update using npm section (@ruyadorno)DEPENDENCIES
3654890fbremove ignored dep (@nlf)a4a0e68a9#3362 check less stuff into node_modules (@isaacs)7d5b049b6#3365 chore(package) Use a "files" list (@isaacs)