Skip to content

Recommend against id as a resource id name#3198

Merged
JordonPhillips merged 2 commits into
mainfrom
resource-id-docs
Jul 1, 2026
Merged

Recommend against id as a resource id name#3198
JordonPhillips merged 2 commits into
mainfrom
resource-id-docs

Conversation

@JordonPhillips

Copy link
Copy Markdown
Contributor

This updates the docs to recommend against using the literal id as a resource shape identifier name, since it can be confusing when child resources are brought into the mix. This also updates examples across the board.

Fixes #3159


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

This updates the docs to recommend against using the literal `id` as
a resource shape identifier name, since it can be confusing when
child resources are brought into the mix. This also updates examples
across the board.

Fixes #3159
@JordonPhillips JordonPhillips requested a review from a team as a code owner June 30, 2026 15:28
@JordonPhillips JordonPhillips requested a review from kstich June 30, 2026 15:28
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@JordonPhillips JordonPhillips merged commit 012250c into main Jul 1, 2026
17 checks passed
@JordonPhillips JordonPhillips deleted the resource-id-docs branch July 1, 2026 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Resource identifier naming: id vs <resource>Id and the sub-resource inheritance trap

2 participants