-
Notifications
You must be signed in to change notification settings - Fork 173
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
feat: Add backwards compatibility validation to package deploy #1909
Conversation
✅ Deploy Preview for zarf-docs canceled.
|
…rsioning for running tests
Add a test case to ensure UnknownVersion doesn't return an error
This reverts commit 7db6b13.
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 think this PR looks good.
I added a comment about a style preference for your consideration. If you prefer it the way you have it now feel free to resolve the 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!
Description
Adds the latest version of Zarf with no breaking package structure changes to build metadata in packages on
package create
Validates the running Zarf CLI version is not less (older) than the last non-breaking version in a package on
package deploy
. Zarf will throw a warning to the user if the CLI version is older than the last non-breaking version to warn before deploying packages that are potentially affected by breaking changes.Related Issue
Fixes #1760
Type of change
Checklist before merging