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

Update codeowners with Cris and Simon #8610

Merged
merged 1 commit into from Jul 4, 2023
Merged

Conversation

simonrw
Copy link
Contributor

@simonrw simonrw commented Jul 4, 2023

As joint service owners, we should be aware of any cloudformation PRs. In particular, as part of the migration to resource providers, there is some configuration in the CloudFormation service directory. We should all be made aware when new resource providers are added to ensure that the centralised configuration is updated.

@simonrw simonrw self-assigned this Jul 4, 2023
@coveralls
Copy link

Coverage Status

coverage: 82.736% (+0.02%) from 82.718% when pulling 34420b8 on cfn/update-codeowners into b411df5 on master.

@simonrw simonrw added the semver: patch Non-breaking changes which can be included in patch releases label Jul 4, 2023
Copy link
Member

@thrau thrau left a comment

Choose a reason for hiding this comment

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

excellent! awesome to see the shared ownership :-) 🚀

@thrau thrau merged commit 2b3fc22 into master Jul 4, 2023
17 of 18 checks passed
@thrau thrau deleted the cfn/update-codeowners branch July 4, 2023 16:10
@dfangl dfangl added this to the 2.2 milestone Jul 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants