Commit d513554
authored
chore: require cli integ test when changing/adding a feature flag. (#34584)
A feature flag has the potential to impact the behavior of all CDK apps, including those defined as part our [CLI integration test package](https://github.com/aws/aws-cdk-cli/tree/main/packages/%40aws-cdk-testing/cli-integ).
For example: #34556
This PR made it so deploying CDK apps with a lambda function now also requires permissions to create log groups. This is not a breaking change for the CDK, but it did broke in our pipeline because of some permission limitations we impose on it.
It could have been caught in the test pipeline.
----
*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*1 parent d8e6c09 commit d513554
1 file changed
+1
-11
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
24 | 24 | | |
25 | 25 | | |
26 | 26 | | |
27 | | - | |
28 | | - | |
29 | | - | |
30 | | - | |
31 | | - | |
32 | | - | |
33 | | - | |
34 | | - | |
35 | | - | |
36 | | - | |
37 | | - | |
| 27 | + | |
38 | 28 | | |
39 | 29 | | |
40 | 30 | | |
| |||
0 commit comments