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: simplify codeowners file #7069

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

janechu
Copy link
Collaborator

@janechu janechu commented Feb 10, 2025

Pull Request

📖 Description

The CODEOWNERS file is too complex and we have a limited number of reviewers at present so this change should help move things through the development process more swiftly while maintaining quality for trusted reviewers.

This change also removes @nicholasrice due to current write permissions not being available for the user which throws an error in GitHub.

✅ Checklist

General

  • I have included a change request file using $ npm run change
  • I have added tests for my changes.
  • I have tested my changes.
  • I have updated the project documentation to reflect my changes.
  • I have read the CONTRIBUTING documentation and followed the standards for this project.

@janechu janechu force-pushed the users/janechu/simplify-codeowners branch from b216bb8 to 7b912e4 Compare February 10, 2025 20:08
@janechu janechu changed the title Simplify codeowners file chore: simplify codeowners file Feb 10, 2025
Copy link
Member

@chrisdholt chrisdholt left a comment

Choose a reason for hiding this comment

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

Approving based on the resolution from the S.C. meeting

@janechu janechu merged commit dd87b12 into main Feb 13, 2025
8 checks passed
@janechu janechu deleted the users/janechu/simplify-codeowners branch February 13, 2025 21:00
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