-
Notifications
You must be signed in to change notification settings - Fork 7
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 flag to support delete changes #258
Conversation
Pull Request Report PR Title ✅ Title follows the conventional commit spec. |
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.
Oli idea for the refactor, other than that lgtm!
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 there something to be made here :)
Proposed changes
Add a flag to show missing resources. By default the Snapshot API will not delete missing resources in the target org unless specified. Similarly, the preview command in the CLI will not show resources to delete unless the user specifically adds the flag to the command.
Example
Without the flag
With the flag
Testing
CDX-361