Skip to content

Script for promoting people to Python triage team. #345

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

Merged
merged 2 commits into from
Aug 10, 2019

Conversation

Mariatta
Copy link
Member

@Mariatta Mariatta commented Aug 1, 2019

Would be nice to have this as a bot.
API documentation: https://developer.github.com/v3/teams/members/#add-or-update-team-membership

Script can be run by an admin/owner of the Python GitHub org.

Copy link

@willingc willingc left a comment

Choose a reason for hiding this comment

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

@Mariatta Looks good to me. Just double checking that "maxking" is not a typo?

# must have repo, and admin:org permissions
gh = GitHubAPI(session, "python", oauth_token=os.getenv("GH_AUTH"), cache=cache)
triage_team = await get_team(gh, "python-triage")
response = await add_to_team(gh, "maxking", triage_team["id"])
Copy link

Choose a reason for hiding this comment

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

Not sure what "maxking" represents but assuming it is some sort of group name.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks! maxking is Abhilash Raj's GitHub username. Since he is being considered for core dev privilege, I figured it would be great to add him to Python triage first. I've also got Barry's approval to add Abhilash.

Copy link

Choose a reason for hiding this comment

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

That's totally cool. Thanks for clarifying.

@Mariatta Mariatta merged commit 68f5002 into master Aug 10, 2019
@Mariatta Mariatta deleted the add-promote-to-triage-script branch August 10, 2019 18:51
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.

3 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy