Skip to content

Backfill all currently listed alumni's teams #1134

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 1 commit into from
Dec 4, 2023

Conversation

dtolnay
Copy link
Member

@dtolnay dtolnay commented Dec 1, 2023

Based on the code from #1125 to extract former team membership from commit history.

This PR fills in team membership for all contributors currently listed in the deprecated alumni.toml, per Ryan's recommendation in #1126 (review) to bring back deleted teams in archived form.

The new archived teams added in this PR need to be filled in with other former members beyond this, but I will handle that separately. For now this PR is enough to bring the deprecated alumni team down to 0 members, and begin removing code related to it.

The only change to the website will be for birkenfeld, who will begin to appear under Clippy alumni at https://www.rust-lang.org/governance/teams/dev-tools#clippy. The other teams are archived teams, which do not get reflected on the website. All archived team members and birkenfeld will continue to appear in the overall alumni list at https://www.rust-lang.org/governance/teams/alumni.

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -24,6 +24,7 @@ alumni = [
"mcarton",
"phansch",
"camsteffen",
"birkenfeld",
Copy link
Member Author

Choose a reason for hiding this comment

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

Copy link
Member

@rylev rylev left a comment

Choose a reason for hiding this comment

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

LGTM - we can address the one comment in a follow up.

"valgrimm",
"whitequark",
]
members = []
# Temporary key that includes all members listed as alumni in the individual
Copy link
Member

Choose a reason for hiding this comment

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

I guess this key is not really temporary as we still want an alumni team that includes all other teams' alumni. Perhaps we should remove this comment?

@rylev rylev merged commit 0bcf02c into rust-lang:master Dec 4, 2023
@dtolnay dtolnay deleted the alumniarchive branch December 4, 2023 08:31
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.

2 participants