Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Release/v7.0.0 beta.12 #1847

Merged
merged 9 commits into from
Sep 22, 2020
Merged

Conversation

ruyadorno
Copy link
Contributor

v7.0.0-beta.12 (2020-09-22)

ruyadorno and others added 7 commits September 22, 2020 12:24
This updates node-gyp to v7, allowing us to deduplicate a lot of
significant dependencies.

Fix: #1281
Fix: npm/statusboard#67

PR-URL: #1846
Credit: @isaacs
Close: #1846
Reviewed-by: @ruyadorno
This is the CLI portion of npm/arborist#136

PR-URL: #1819
Credit: @isaacs
Close: #1819
Reviewed-by: @ruyadorno
We are targetting 100% test coverage for npm v7 GA.  Using a
coverage-map, we require that any of the newly created tests fully cover
the module that they are testing.  However, this has the side effect of
_always_ hitting 100% coverage when running 'npm test', even though not
all modules are being tested.

This adds a new 'load-all' test which, in 'check-coverage' mode, tells
nyc to include every file in the project.

This also removes the double-run of our tests in CI, where we run once
and then immediately run the same exact thing again for Coveralls, and
sends the 'check-coverage' output to Coveralls instead.

PR-URL: #1820
Credit: @isaacs
Close: #1820
Reviewed-by: @ruyadorno
PR-URL: #1699
Credit: @luciomartinez
Close: #1699
Reviewed-by: @ruyadorno
@ruyadorno ruyadorno requested a review from a team as a code owner September 22, 2020 17:01
@ruyadorno ruyadorno merged commit 9bc6966 into release/v7.0.0-beta Sep 22, 2020
@darcyclarke darcyclarke added this to the OSS - Sprint 15 milestone Sep 25, 2020
@nlf nlf deleted the release/v7.0.0-beta.12 branch March 28, 2022 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants