Skip to content

feat(core): Validations class is the new way to add validation plugins to CDK Apps#37611

Merged
mergify[bot] merged 17 commits intomainfrom
conroy/validations
Apr 21, 2026
Merged

feat(core): Validations class is the new way to add validation plugins to CDK Apps#37611
mergify[bot] merged 17 commits intomainfrom
conroy/validations

Conversation

@kaizencc
Copy link
Copy Markdown
Contributor

@kaizencc kaizencc commented Apr 16, 2026

Reason for this change

Part of aws/aws-cdk-rfcs#899. Depends on #37613. Validations is the new one-stop shop for post-synthesis validation. This PR starts with the addPlugins() API that mirrors the functionality of supplying policyValidationBeta1 to your CDK App today.

Description of how you validated changes

Existing tests succeed and new tests verify that Validations.addPlugin() does the same thing.

Checklist


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 April 16, 2026 03:33
@github-actions github-actions Bot added the p2 label Apr 16, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team April 16, 2026 03:33
@mergify mergify Bot added the contribution/core This is a PR that came from AWS. label Apr 16, 2026
Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(This review is outdated)

@kaizencc kaizencc changed the title feat(core): Validations class is the new way to add validation plugins to CDK Apps feat(core): Validations class is the new way to add validation plugins to CDK Apps Apr 16, 2026
@kaizencc kaizencc changed the base branch from main to conroy/remove-beta1 April 16, 2026 17:40
@kaizencc kaizencc added the pr-linter/exempt-integ-test The PR linter will not require integ test changes label Apr 16, 2026
The merge from main duplicated full-body deprecated interfaces
(from #37613) alongside the extends-based aliases that this PR
uses. Restored report.ts and validation.ts to their correct
pre-merge state.
The merge from main kept both the full-body deprecated Beta1
interfaces (from #37613) and the extends-based aliases from
this branch, causing duplicate identifiers. Fixed by keeping
the #37613 full-body approach with correct Beta1 names and
removing the extends-based aliases.
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 21, 2026

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).

@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 21, 2026

Merge Queue Status

  • Entered queue2026-04-21 21:08 UTC · Rule: default-squash
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-21 21:08 UTC · at e992799db197704f92a41dedccf82f9747d6b04d · squash

This pull request spent 12 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit 95696b4 into main Apr 21, 2026
19 of 20 checks passed
@mergify mergify Bot deleted the conroy/validations branch April 21, 2026 21:08
@github-actions
Copy link
Copy Markdown
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 21, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2 pr-linter/exempt-integ-test The PR linter will not require integ test changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants