-
Notifications
You must be signed in to change notification settings - Fork 332
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
v17 #1345
v17 #1345
Conversation
Several dependencies dropped support
--global was erroneously deprecated in node v16.15.1 and npm v8.12.0 on Windows. npm/cli#4980 npm/cli#4982
Several dependencies dropped support
I rebased v17 on main and pushed to v17-next in case you are having trouble with that. |
--global was erroneously deprecated in node v16.15.1 and npm v8.12.0 on Windows. npm/cli#4980 npm/cli#4982
getPackageVersionProtected does catch any errors, but it rethrows them if they are not 404's.
I've merged |
README.md
Outdated
@@ -332,7 +332,7 @@ Options that take no arguments can be negated by prefixing them with `--no-`, e. | |||
</tr> | |||
<tr> | |||
<td>--root</td> | |||
<td>Runs updates on the root project in addition to specified workspaces. Only allowed with <code>--workspace</code> or <code>--workspaces</code>. (default: false)</td> | |||
<td>Runs updates on the root project in addition to specified workspaces. Only allowed with <code>--workspace</code> or <code>--workspaces</code>. (default: true) (default: true)</td> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bug?
And lockfile-lint's engine does not reflect the dependency's engine.
7ef0d7b
to
4ab2c9d
Compare
There is more work to do to upgrade dependencies and convert to esm, but I'm going to get this merged so that we can work off Legacy v16 code will be kept on branch v16 for emergency patches. |
Intro and discussion: #1346