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: add code owner permissions to change directory #6882

Merged
merged 4 commits into from
Jan 4, 2024

Conversation

scomea
Copy link
Collaborator

@scomea scomea commented Jan 2, 2024

📖 Description

Code owners that own individual packages can't approve PR's that include change files at the project root. This change gives Owners of the "fast-foundation" package need permissions to the root 'change' folder as well in order to approve pr's with change files.

🎫 Issues

ad-hoc

✅ Checklist

General

  • I have included a change request file using $ yarn 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.

Component-specific

.github/CODEOWNERS Outdated Show resolved Hide resolved
@chrisdholt
Copy link
Member

One change we should try - add the directory as the last line in this file with no owners. That should tell GH that there are no owners required for those files.

@scomea
Copy link
Collaborator Author

scomea commented Jan 4, 2024

@microsoft-github-policy-service agree company="Qualcomm Innovation Center, inc."

@chrisdholt chrisdholt merged commit da58013 into microsoft:master Jan 4, 2024
5 checks passed
@scomea scomea deleted the users/scomea/code-owners branch January 4, 2024 18:02
janechu pushed a commit that referenced this pull request Jun 10, 2024
* edit change dir

* tweak

* more change owners

* no change owners
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