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 owner #2927

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Conversation

JimSuplizio
Copy link
Member

Specifically give the CODEOWNERS file an owner.

@JimSuplizio JimSuplizio requested a review from a team September 24, 2024 18:00
@JimSuplizio JimSuplizio self-assigned this Sep 24, 2024
@RickWinter
Copy link
Member

Can you add Ronnie as an owner on that file as well

@JimSuplizio
Copy link
Member Author

Can you add Ronnie as an owner on that file as well

Yup. The commit I just pushed added him.

@JimSuplizio
Copy link
Member Author

The linter pipeline passed which is the only thing testing this change. I'm going to merge without waiting for the other pipelines.

@JimSuplizio JimSuplizio merged commit dfced24 into Azure:main Sep 24, 2024
23 of 25 checks passed
@JimSuplizio JimSuplizio deleted the UpdateCodeownersOwner branch September 24, 2024 18:29
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