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

feat: Custom GitHubActionRole subject claims #753

Merged
merged 12 commits into from
Oct 30, 2023

Conversation

hertzsprung
Copy link
Contributor

@hertzsprung hertzsprung commented Oct 14, 2023

Adds custom subject claims for #739

Closes #739

@hertzsprung hertzsprung changed the title Custom GitHubActionRole subject claims feat:custom GitHubActionRole subject claims Oct 14, 2023
@hertzsprung hertzsprung changed the title feat:custom GitHubActionRole subject claims feat: Custom GitHubActionRole subject claims Oct 14, 2023
Copy link
Contributor

@kaizencc kaizencc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @hertzsprung this looks great! Thanks for your contribution. The only thing I need here is an update to the readme to introduce this new functionality there :)

@hertzsprung
Copy link
Contributor Author

Thanks for the reminder about the README, I've documented subjectClaims in the README now.

@hertzsprung
Copy link
Contributor Author

@kaizencc let me know if there's anything else you need, cheers!

Copy link
Contributor

@kaizencc kaizencc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @hertzsprung! Really appreciate the contribution

@mergify mergify bot merged commit aee0488 into cdklabs:main Oct 30, 2023
12 checks passed
@hertzsprung hertzsprung deleted the 739-custom-subject-claims branch November 1, 2023 16:11
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.

Allow GitHubActionRoleProps to specify custom claims
2 participants