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: set wg-releases as CODEOWNER for AppVeyor configs #41485

Merged
merged 1 commit into from Mar 1, 2024

Conversation

dsanders11
Copy link
Member

Description of Change

Follow-up to #41470, which couldn't open a PR when run due to the reviewer team not being found. Looks like this is because the bot would need perms to read the org teams. Rather than give the bot additional permissions, it seems more reasonable to simply mark those files as owned by the team in CODEOWNERS, which should accomplish the same thing.

Checklist

  • PR description included and stakeholders cc'd

Release Notes

Notes: none

@dsanders11 dsanders11 requested review from VerteDinde and a team February 29, 2024 16:47
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Feb 29, 2024
@VerteDinde VerteDinde removed the new-pr 🌱 PR opened in the last 24 hours label Feb 29, 2024
@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels Feb 29, 2024
@jkleinsc
Copy link
Contributor

jkleinsc commented Mar 1, 2024

Merging as CI failure unrelated to PR change.

@jkleinsc jkleinsc merged commit 9565b7d into main Mar 1, 2024
22 of 23 checks passed
@jkleinsc jkleinsc deleted the appveyor-config-codeowners branch March 1, 2024 20:53
Copy link

release-clerk bot commented Mar 1, 2024

No Release Notes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants