-
Notifications
You must be signed in to change notification settings - Fork 16
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
Allow filtering by version level #534
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Nice result!
As a side node, I thought the UI was more robust in the past, didn't t check for unknown dependencies? Now it doesn't seem to warn in case of misspelled options and/or arguments:
tezos-developer-docs % npm run check-dependencies -- --blabla zvelt
lists all the outdated dependecies, without complaining about anything.
In this case you are passing an argument |
3ad5846
to
ec866c7
Compare
Added error checking so it will fail if you pass an unknown |
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.
LGTM
The current dependency script alerts on any version difference up to the fixpack level. This change allows you to pass
--major
to show only places where dependencies are out of date by the major version or--minor
for minor version.For example,
npm run check-dependencies -- --major octez
prints this:But
npm run check-dependencies -- --minor octez
prints this: