Age | Commit message (Collapse) | Author |
|
|
|
|
|
|
|
|
|
|
|
BREAKING CHANGE: all workspace packages are now compatible with the
following semver range for node: `^14.17.0 || ^16.13.0 || >=18.0.0`
|
|
This also replaces the previous check for known broken versions of node
with an exception handler for syntax errors in order to try and give a
nicer error message when attempting to run npm on older node versions.
BREAKING CHANGE: `npm` is now compatible with the following semver range
for node: `^14.17.0 || ^16.13.0 || >=18.0.0`
Ref: npm/statusboard#519
|
|
|
|
|
|
|
|
|
|
Both nodejs and npm branches are now "main"
|
|
|
|
|
|
|
|
* chore: update bot account and secrets for nodejs/node updates
* chore: update bot account and secrets for npm/benchmarks triggers
* chore: add npm CLI robot to mailmap and exclude from AUTHORS
|
|
|
|
|
|
The updates the issue comment check in the benchmark script, so it only
will trigger if posted by a user with admin rights to the repo. It also
changes the comment to trigger it to `@npm-robot benchmark this`
For pull requests, it now limits benchmarks only when the files changed
include something inside `lib/`.
Requires https://github.com/npm/benchmarks/pull/30 to be merged in the
benchmark repo to allow `issue_comment` as a valid trigger.
|
|
|
|
You cannot use a secret as part of an if expression in actions
|
|
|
|
|
|
|
|
|
|
|
|
|
|
- add some basic tests for docs
- make dockhand script work on windows
- cleanup main CI to match template-oss
- add a git status check for cli ci tests
- use resetdeps for ci steps
|
|
- `no-shadow` was disabled for the cli and arborist due to the amount
of changes required. These should be fixed later.
- Manually updated linting in the cli root to match closer to what
template-oss does
- Included docs to be managed with template-oss. This required adding
an empty test for now, with the plan to add real tests later.
|
|
* fix: 100% coverage in tests
* Removed dead code in `lib/utils/usage.js`.
* Removed dead code in `lib/base-command.js`.
* Removed "load-all" test, we currently have 100% coverage and new PRs
without tests will be rejected if they don't add coverage for new
files.
* Removed `check-coverage` script as a separate command.
* Removed separate coverage test in ci.yml.
* Removed `coverage` flag from tap config, the default is already to
enforce 100% coverage.
Removed a tiny bit of dead code resulting from this
* fix: clean up usage output
Removed usage lib, rolled logic into base-command.js
Cleaned up usage output to be less redundant
|
|
Should get release-please working in workspaces
|
|
|
|
|
|
- updated template-oss and manually added arborist ci
- updated arborist ci to include path
- fixed workspace reference and upgraded template-oss
- upgraded template-oss properly
|
|
Guiding how to update the environment in bug pull request much clear.
|
|
Added libnpm workspaces and arborist
|
|
PR-URL: https://github.com/npm/cli/pull/4145
Credit: @nlf
Close: #4145
Reviewed-by: @wraithgar
|
|
PR-URL: https://github.com/npm/cli/pull/4118
Credit: @felipeplets
Close: #4118
Reviewed-by: @darcyclarke
|
|
PR-URL: https://github.com/npm/cli/pull/4050
Credit: @fritzy
Close: #4050
Reviewed-by: @wraithgar
PR-URL: https://github.com/npm/cli/pull/4050
Credit: @fritzy
Close: #4050
Reviewed-by: @wraithgar
|
|
Also update with other items from template-oss
PR-URL: https://github.com/npm/cli/pull/3982
Credit: @lukekarrys
Close: #3982
Reviewed-by: @wraithgar
PR-URL: https://github.com/npm/cli/pull/3982
Credit: @lukekarrys
Close: #3982
Reviewed-by: @wraithgar
PR-URL: https://github.com/npm/cli/pull/3982
Credit: @lukekarrys
Close: #3982
Reviewed-by: @wraithgar
|
|
PR-URL: https://github.com/npm/cli/pull/3958
Credit: @nlf
Close: #3958
Reviewed-by: @wraithgar
|
|
We don't need duplicate builds on pull requests and pushes. This should
cut our actions essentially down to half of what they are now.
PR-URL: https://github.com/npm/cli/pull/3981
Credit: @wraithgar
Close: #3981
Reviewed-by: @darcyclarke
|
|
Adds a new required checkbox to our bug template directing people to use
the latest npm version to verify their bug still exists.
PR-URL: https://github.com/npm/cli/pull/3977
Credit: @wraithgar
Close: #3977
Reviewed-by: @darcyclarke
|
|
PR-URL: https://github.com/npm/cli/pull/3857
Credit: @darcyclarke
Close: #3857
Reviewed-by: @wraithgar
|
|
PR-URL: https://github.com/npm/cli/pull/3858
Credit: @darcyclarke
Close: #3858
Reviewed-by: @wraithgar
|
|
BREAKING CHANGE:
- Drop official support for node versions less than v12.
- Drop support for `require('npm')`
- Update a few subdependencies that dropped node10 support, and brought
in the latest node-gyp
PR-URL: https://github.com/npm/cli/pull/3762
Credit: @wraithgar
Close: #3762
Reviewed-by: @fritzy
|
|
for general support we are now direct folks to github.community not general
npm support
PR-URL: https://github.com/npm/cli/pull/3850
Credit: @MylesBorins
Close: #3850
Reviewed-by: @wraithgar
|
|
When we accidentally edit the auto-generated portions of the docs, this
will catch the error and cause CI to fail.
Later phase automated safety check that the early-stage human commenting
in the last commit also addresses.
Re: #3654
Re: #3630
PR-URL: https://github.com/npm/cli/pull/3655
Credit: @isaacs
Close: #3655
Reviewed-by: @nlf
|
|
Currently, the name: `npm-robot` is being used in nodejs changelogs as
the atttribution author name for npm-update commits. This makes it so
that entries in the changelog referring to our updates reads like:
```
upgrade npm to 7.18.1 (npm-robot) #39065
```
This change makes it so that the name used in commits is `npm team`,
this way the changelogs entry should read instead as:
```
upgrade npm to 7.18.1 (npm team) #39065
```
Ref: https://nodejs.org/en/blog/release/v16.4.0/
PR-URL: https://github.com/npm/cli/pull/3541
Credit: @ruyadorno
Close: #3541
Reviewed-by: @wraithgar, @targos, @BethGriggs
|
|
- Setup ./packages/* as workspaces
- Moved source from: https://github.com/npm/libnpmdiff to
./packages/libnpmdiff
- Added CI target for workspaces
Relates to: https://github.com/npm/statusboard/issues/362
PR-URL: https://github.com/npm/cli/pull/3386
Credit: @ruyadorno
Close: #3386
Reviewed-by: @wraithgar
|