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

chore: remove buildkite codeowners #7340

Merged
merged 2 commits into from
Aug 4, 2022

Conversation

matklad
Copy link
Contributor

@matklad matklad commented Aug 4, 2022

@matklad matklad requested a review from a team as a code owner August 4, 2022 10:40
@matklad matklad requested review from nikurt and akhi3030 August 4, 2022 10:40
@matklad
Copy link
Contributor Author

matklad commented Aug 4, 2022

@akhi3030 I don't remember, did we figured out what's the ownership story for our CI is?

@akhi3030
Copy link
Collaborator

akhi3030 commented Aug 4, 2022

I didn't realise we could just delete the whole file. I thought we would just need to remove the following line.

/.buildkite/ @rtsai123 @mhalambek

@akhi3030 I don't remember, did we figured out what's the ownership story for our CI is?

The only progress we made really is proposing that nearcore-codeowners own it for now. Longer term, we may need to pick a subset of the protocol team to own it.

@matklad matklad marked this pull request as draft August 4, 2022 14:39
@matklad matklad force-pushed the m/no-buildkit-codeowners branch from 225059a to 7f2c9c1 Compare August 4, 2022 14:47
@matklad matklad marked this pull request as ready for review August 4, 2022 14:48
@matklad
Copy link
Contributor Author

matklad commented Aug 4, 2022

cc @rtsai123

@near-bulldozer near-bulldozer bot merged commit 78ef2f5 into near:master Aug 4, 2022
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.

3 participants