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

🤖 Fix: CODEOWNERS #10

Merged
merged 2 commits into from
Jan 7, 2025

Conversation

joris974
Copy link
Member

This PR adds the required CODEOWNERS configuration to meet repository compliance standards.

@chris-martin chris-martin enabled auto-merge (squash) December 19, 2024 18:14
@chris-martin chris-martin requested review from a team and tomcarste and removed request for a team January 3, 2025 00:35
@StevenXL
Copy link

StevenXL commented Jan 7, 2025

@joris974 should this be merged (or maybe closed)?

@joris974
Copy link
Member Author

joris974 commented Jan 7, 2025

@joris974 should this be merged (or maybe closed)?

Yes this needs to be merged. But branch protection is preventing 1) me from merging as I am the author, 2) Chris approving because they are the author of the last commit.
So another person needs to approve for this to be merge-able

@joris974 joris974 disabled auto-merge January 7, 2025 19:01
@joris974 joris974 enabled auto-merge (squash) January 7, 2025 19:01
@joris974 joris974 force-pushed the automated-repo-fix/fix/codeowners-1734033202181 branch from aca3450 to 28984c0 Compare January 7, 2025 19:02
@joris974 joris974 merged commit 23216ec into main Jan 7, 2025
8 checks passed
@joris974 joris974 deleted the automated-repo-fix/fix/codeowners-1734033202181 branch January 7, 2025 19:10
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.

4 participants