-
Notifications
You must be signed in to change notification settings - Fork 14.4k
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
Switch to finer controls on approver access #42206
Conversation
sftim
commented
Jul 25, 2023
- Add a new website owners group
- Separate out “I can approve changes to English” from “I can approve changes anywhere on the website”
- Add labels to PRs that make web development changes
- Let the localization team review changes that add new localizations (?)
- Don't allow English reviewers to review changes for other languages (website owners still can)
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify site configuration. |
af60b6d
to
70d16f5
Compare
I need to make time to work on this. |
Relevant to #45762 |
121db8e
to
4e40dfa
Compare
4e40dfa
to
b51b327
Compare
b51b327
to
6e83b09
Compare
/retitle Switch to finer controls on approver access |
/remove-sig release |
sig-docs-localization-reviewers: # PR reviews for localization changes | ||
- a-mccarthy | ||
- divya-mohan0209 | ||
- natalisucks | ||
- nate-double-u | ||
- reylejano | ||
- sftim | ||
- seokho-son |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All new (but likely same as localization owners)
This is required to allow changes to the Hugo configuration.
/lgtm This looks good to me, however, I'd like for this change to take place after the v1.30 release (currently slated for April 17, 2024), so that no behaviour changes for the Docs Lead and Release Lead (cc. @drewhagen @katcosgrove). Thanks for this, @sftim 🤝 |
LGTM label has been added. Git tree hash: b0f7866fe69e318a0304c3002bd8506ea73b313d
|
- drewhagen # RT 1.30 Docs Lead | ||
- katcosgrove # RT 1.30 Lead |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Moved, not removed
This change makes sense to me, the en owners group should only be allowed to approve en PRs while website-owners have the ability to approve other localizations /approve With 1.30 released yesterday (April 17, 2024), I'm unholding this PR |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: reylejano The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |