-
Notifications
You must be signed in to change notification settings - Fork 181
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
Generate github teams according to RFC-0005 #310
Conversation
stephanme
commented
Jun 15, 2022
- teams for WG, WG areas, WG leads and WG bots
- added empty bot user list to all WG yamls
- enabled only for ARD and FI WGs as pilots
- teams for WG, WG areas, WG leads and WG bots - added empty bot user list to all WG yamls - enabled only for ARD and FI WGs as pilots
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.
How will this script deal with repos outside the cloudfoundry org?
Co-authored-by: Ruben Koster <[email protected]>
Ignore all repos from other github orgs listed in WG yaml. E.g. FI WG. Fixed bot users in FI WG as all github users have name and github id WG yaml (also bots).
TOC.md and WG md files which contain yaml blocks processed by org-management.py
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.
Since it filters for repo's in the cloudfoundry org all my concerns are addressed.
maintainers = {u["github"] for u in wg["execution_leads"]} | ||
maintainers |= {u["github"] for u in wg["technical_leads"]} | ||
approvers = {u["github"] for a in wg["areas"] for u in a["approvers"]} | ||
repositories = {r for a in wg["areas"] for r in a["repositories"] if r.startswith("cloudfoundry/")} |
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.
We will have to rework this line later when we onboard other orgs into the automation but for now it should be fine.