-
Notifications
You must be signed in to change notification settings - Fork 16
Allow filtering by version level #534
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
NicNomadic
left a comment
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 |
NicNomadic
left a comment
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
--majorto show only places where dependencies are out of date by the major version or--minorfor minor version.For example,
npm run check-dependencies -- --major octezprints this:But
npm run check-dependencies -- --minor octezprints this: