feat(cdk): cdk diff --quiet suppresses progress messages#26652
feat(cdk): cdk diff --quiet suppresses progress messages#26652mergify[bot] merged 3 commits intoaws:mainfrom
cdk diff --quiet suppresses progress messages#26652Conversation
aws-cdk-automation
left a comment
There was a problem hiding this comment.
The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.
A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.
|
|
Adds support for the --quiet flag within cdk diff command, for users that would not like the following messages 'Stack <name>' and 'There were no differences' to be silenced for stacks which have identified differences
7799b62 to
7d0cc38
Compare
rix0rrr
left a comment
There was a problem hiding this comment.
Thanks for this! Looks well-done!
cdk diff --quiet only prints differences
cdk diff --quiet only prints differencescdk diff --quiet prints nothing on no difference
cdk diff --quiet prints nothing on no differencecdk diff --quiet suppresses diff fluff
✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.
cdk diff --quiet suppresses diff fluffcdk diff --quiet suppresses progress messages
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
Hello maintainers
This is my first PR, so all feedback is welcome. This change adds support for the
--quietflag within cdk diff command, for users that would not like the following messages 'Stack ' and 'There were no differences' to be silenced for stacks which do not have any identified differences. The request was made by @nomike in the #26526, I simply followed the very well detailed description.The change follows similar structure to existing command and how they are passed, I have also added to the README as per the contributing guidelines.
Before submitting I have.
@aws-cdk-testing/cli-integ/bin/run-suite -a cli-integ-testsCloses #26526
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license