Skip to content

Conversation

rix0rrr
Copy link
Contributor

@rix0rrr rix0rrr commented Aug 12, 2025

This change goes together with
aws/aws-cdk-cli#782, to undo some decisions made in https://github.com/aws/aws-cdk/pull/35108/files and aws/aws-cdk-cli#742.

We used to store unconfiguredBehavesLike values for both v1 and v2 (and v3, v4, etc) into the feature flag report, but we only needed to store the value for the current library version. We are now defining the "v2" field to hold the current version information, and are updating the feature flag generation code to not emit v1 data, and stay correct if we ever start releasing v3.

This is labeled as a chore and not a fix because it doesn't really fix user-visible behavior.


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

…ry versions

This change goes together with
aws/aws-cdk-cli#782, to undo some decisions made
in https://github.com/aws/aws-cdk/pull/35108/files and
aws/aws-cdk-cli#742.

We used to store `unconfiguredBehavesLike` values for both `v1` and `v2`
(and `v3`, `v4`, etc) into the feature flag report, but we only needed to
store the value for the *current* library version. We are now defining
the "v2" field to hold the current version information, and are updating
the feature flag generation code to not emit `v1` data, and stay correct
if we ever start releasing `v3`.
@rix0rrr rix0rrr requested a review from a team as a code owner August 12, 2025 08:51
@aws-cdk-automation aws-cdk-automation requested a review from a team August 12, 2025 08:51
@github-actions github-actions bot added the p2 label Aug 12, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Aug 12, 2025
Copy link
Contributor

mergify bot commented Aug 12, 2025

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

Copy link
Contributor

mergify bot commented Aug 12, 2025

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 mergify bot merged commit a67c875 into main Aug 12, 2025
18 checks passed
@mergify mergify bot deleted the huijbers/cloud-assembly-field-v2 branch August 12, 2025 15:52
Copy link
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 Aug 12, 2025
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants