Skip to content

fix(core): tree metadata does not contain logical ID#37630

Open
rix0rrr wants to merge 1 commit intomainfrom
huijbers/logicalid-metadata
Open

fix(core): tree metadata does not contain logical ID#37630
rix0rrr wants to merge 1 commit intomainfrom
huijbers/logicalid-metadata

Conversation

@rix0rrr
Copy link
Copy Markdown
Contributor

@rix0rrr rix0rrr commented Apr 17, 2026

The metadata in tree.json contains a lot of information about L1s (resource types and properties), but is missing the logical IDs.

Add those.


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

The metadata in `tree.json` contains a lot of information about L1s
(resource types and properties), but is missing the logical IDs.

Add those.
@rix0rrr rix0rrr requested a review from a team as a code owner April 17, 2026 14:23
@github-actions github-actions bot added the p2 label Apr 17, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team April 17, 2026 14:24
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Apr 17, 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.

The pull request linter fails with the following errors:

❌ Fixes must contain a change to an integration test file and the resulting snapshot.

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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants