-
Notifications
You must be signed in to change notification settings - Fork 704
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
New page for pub bump #6196
New page for pub bump #6196
Conversation
Visit the preview URL for this PR (updated for commit 765695e): |
### `-n, --dry-run` | ||
|
||
Reports what would change for the version number, and shows the version diff, | ||
without changing anything. |
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.
This is not actually implemented (perhaps it should be though...)
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.
Oh dry-run
isn't implemented? It comes up a few times in the PR though, is that just like making space for it in the future?
I'll remove it anyway for now to land the page, just curious!
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.
Oh I'm sorry - you are totally right.
I was looking at dart pub bump --help
but should of course look at dart pub bump <subcommand> --help
.
And it is indeed implemented (by me...).
Thanks for pointing this out!
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.
No worries! Thanks for explaining, I put it back and specified it's for each subcommand
Co-authored-by: Sigurd Meldgaard <[email protected]>
Fixes #6147
Not sure if I wrote line 9 correctly:
$ dart pub bump <subcommand> [arguments]
Let me know if any other info would be helpful to include!