-
-
Notifications
You must be signed in to change notification settings - Fork 15
Add dry_run
input option
#204
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
Conversation
This is useful if you don't actually want to release, but you're just interested in whether or not a release would be created. Useful to gate a secondary job with an environment, without creating requests for each and every change. Fixes release-plz#200
If the input `command` is left unspecified (and so both `release` and `release-pr` are ran), | ||
the `--dry-run` flag is only added to the `release` command | ||
(the flag isn't added to the `release-pr` command). | ||
Useful if you're only interested in whether or not a release (pr) would be created." |
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.
how do you determine if a release-pr will be created? can you explain more?
can you also open a PR for https://github.com/release-plz/release-plz/blob/main/website/docs/github/input.md ? |
- uses: release-plz/[email protected]
with:
command: release-dryrun I know this doesn't cleanly map to CLI Another alternative is to just have an "untyped" - uses: release-plz/[email protected]
with:
command: release
options: --dry-run FYI I'm interested in this (or rather #200) since a conditional |
I prefer having the 2 inputs with the 4 combinations instead of |
This is useful if you don't actually want to release, but you're just interested in whether or not a release would be created. Useful to gate a secondary job with an environment, without creating requests for each and every change.
Fixes #200