Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand allowed tagging arn member names #2528

Merged
merged 1 commit into from
Feb 19, 2025
Merged

Expand allowed tagging arn member names #2528

merged 1 commit into from
Feb 19, 2025

Conversation

kstich
Copy link
Contributor

@kstich kstich commented Feb 18, 2025

Allows for the resource name component, the arn name component, or both name components to be specified. Empty names, non-matching names, or names that have repeat components will fail.

Other minor interface cleanup is included.


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

@kstich kstich requested a review from a team as a code owner February 18, 2025 10:07
@kstich kstich requested a review from joewyz February 18, 2025 10:07
@kstich kstich force-pushed the tag_arn_member_fix branch from 8954dde to 0080c21 Compare February 18, 2025 10:13
Allows for the resource name component, the arn name component, or
both name components to be specified. Empty names, non-matching names,
or names that have repeat components will fail.
@kstich kstich force-pushed the tag_arn_member_fix branch from 0080c21 to e871dec Compare February 18, 2025 18:40
@kstich kstich merged commit 2bf9b48 into main Feb 19, 2025
8 checks passed
@kstich kstich deleted the tag_arn_member_fix branch February 19, 2025 16:30
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.

2 participants