-
Notifications
You must be signed in to change notification settings - Fork 58
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
Add --from-stage option to version create #1973
base: main
Are you sure you want to change the base?
Add --from-stage option to version create #1973
Conversation
cf9c075
to
ab0a5c2
Compare
ab0a5c2
to
85865f7
Compare
# then do not re-deploy the artifacts or touch the stage | ||
if from_stage: | ||
# verify package exists: | ||
show_obj_row = self.get_existing_app_pkg_info() |
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.
nit: we can directly check self.get_existing_app_pkg_info()
in the if statement
@@ -25,6 +25,7 @@ | |||
* `snow app release-directive set` | |||
* `snow app release-directive unset` | |||
* `snow app version create` now returns version, patch, and label in JSON format. | |||
* Add `--from-stage` flag to `snow app version create` to allow version creation from the content of the stage without re-syncing to the stage. |
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.
Out of curiosity, what does "re-syncing to the stage" mean?
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.
uploading files to the stage from local workspace.
@@ -1923,6 +1923,13 @@ | |||
| untracked or stages | | |||
| files in git. Default: | | |||
| unset. | | |||
| --from-stage When enabled, the | |
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.
I'm curious what is this file?
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.
It contains snapshots from all the commands through unit tests - it is auto-generated, and helps verify that the command output changes (--help) are as expected.
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.
got it thanks
Pre-review checklist
Changes description
Add --from-stage option to version create
When
--from-stage
is provided,snow app version create --from-stage
will create a version from the content of the stage directly without resyncing the local files to the stage.