Skip to content

chore(core): cdk cli handles terminal output for validation report#38158

Open
kaizencc wants to merge 1 commit into
mainfrom
conroyka/remove-validation-pretty-print
Open

chore(core): cdk cli handles terminal output for validation report#38158
kaizencc wants to merge 1 commit into
mainfrom
conroyka/remove-validation-pretty-print

Conversation

@kaizencc

Copy link
Copy Markdown
Contributor

In conjunction with a CLI change aws/aws-cdk-cli#1633, the CLI now handles terminal output for validation. the core library handles writing the validation report as part of the cloud assembly only. It always writes validation-report.json, and will write the legacy policy-validation-report.json if a flag is set.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@kaizencc kaizencc requested a review from a team as a code owner June 16, 2026 20:00
@kaizencc kaizencc changed the title chore(core): CLI handles terminal output for validation report chore(core): cdk cli handles terminal output for validation report Jun 16, 2026
@github-actions github-actions Bot added the p2 label Jun 16, 2026
@mergify mergify Bot added the contribution/core This is a PR that came from AWS. label Jun 16, 2026
@mergify mergify Bot temporarily deployed to automation June 16, 2026 20:00 Inactive
@mergify mergify Bot temporarily deployed to automation June 16, 2026 20:01 Inactive
@github-actions

github-actions Bot commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

⚠️ This pull request description does not follow the correct template structure.

PRs without a linked issue will receive lower priority for review and merging. Please update the description to follow the PR template and include a line like Closes #123 in the Issue section. If no existing issue matches your change, create one first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. p2 pr/needs-maintainer-review This PR needs a review from a Core Team Member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants