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

feat: Add a check to ensure the committers-at-large team has access. #584

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Jan 7, 2025

This tema should have write access to all public repos. They can do any
maintenance work in any public repo that is a part of the Open edX
platform.

@feanil feanil requested a review from kdmccormick January 7, 2025 17:12
@feanil feanil force-pushed the feanil/committers-at-large branch from 9119b86 to 5ef1ca8 Compare January 7, 2025 17:15
This tema should have write access to all public repos.  They can do any
maintenance work in any public repo that is a part of the Open edX
platform.
@feanil feanil force-pushed the feanil/committers-at-large branch from 5ef1ca8 to b54c6ab Compare January 8, 2025 18:42
"""

def __init__(self, api, org, repo):
team = "committers-maintainers-at-large"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is a mouthful but I felt that having it just be committers-at-large could be misconstrued as these users can merge whatever they want and not just maintenance related work.

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.

1 participant