-
Notifications
You must be signed in to change notification settings - Fork 452
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
Add a codeower to important files under .github/workflows #5312
Conversation
Thanks @tkoyama010! I was just looking up how to do this. I'm a little concerned about noise from dependable PRs though |
I'd like to go ahead and merge this but then raise an offline conversation, perhaps on Slack to see if there are other critical parts of code that people want to volunteer to "own". Some potential files:
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #5312 +/- ##
=======================================
Coverage 96.20% 96.20%
=======================================
Files 134 134
Lines 22712 22712
=======================================
Hits 21850 21850
Misses 862 862 |
Co-authored-by: Bane Sullivan <banesullivan@gmail.com>
Overview
@banesullivan I added you to the
testing-and-deployment.yml
codeowner. If we merge this, we cannot merge thetesting-and-deployment.yml
changes without your approval.We can add any other workflows you wish to add.
Details