Sync organization members #1868
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pietroalbini
requested review from
Mark-Simulacrum,
jdno and
marcoieni
as code owners
Dry-run check results |
|
It's removing rust-lang-owner and other bots |
marcoieni
force-pushed
the
sync-org-members
branch
from
1bbac75 to
b8049db
Compare
|
I tried pushing one commit that also updated master, but failed miserably, so I reverted it 😅 |
marcoieni
force-pushed
the
sync-org-members
branch
from
b8049db to
3e935a6
Compare
|
Now Kobzol is not in the authors of the commit anymore. Sorry about that but that's the maximum I can get with my git skills 😂 |
marcoieni
force-pushed
the
sync-org-members
branch
2 times, most recently
from
ebdec80 to
e05929e
Compare
marcoieni
force-pushed
the
sync-org-members
branch
2 times, most recently
from
b8a431b to
c9d7993
Compare
tshepang
reviewed
Jul 15, 2025
marcoieni
force-pushed
the
sync-org-members
branch
from
c9d7993 to
e76e204
Compare
marcoieni
force-pushed
the
sync-org-members
branch
from
e76e204 to
6a05230
Compare
Kobzol
reviewed
Jul 15, 2025
|
|
||
| # GitHub accounts that are allowed to stay in the orgs, | ||
| # even if they may not be members of any team. | ||
| allowed-org-members = [ |
There was a problem hiding this comment.
nit: allowed makes it sound like only these org members may be a part of an org :D Which of course doesn't make sense, and the comment does clarify it. But I don't have a great alternative. Maybe just special-org-members? But it doesn't really matter.
| "rustbot", | ||
|
|
||
| # Infra admins. | ||
| "jdno", |
There was a problem hiding this comment.
Hmm, we start having a bunch of places where we hardcode infra-admins. Maybe it would be better if we loaded the actual contents of the infra-admins team (so that we hardcode only the team in code, but not its members) in code and ignored them?
| use indexmap::IndexMap; | ||
| use serde::{Deserialize, Serialize}; | ||
|
|
||
| pub static BASE_URL: &str = "https://team-api.infra.rust-lang.org/v1"; | ||
|
|
||
| #[derive(Debug, Clone, Serialize, Deserialize, PartialEq, Default)] |
There was a problem hiding this comment.
I don't think we need to expose this to the JSON API? It's an internal implementation detail. Can't you just pass &Config from team to sync-team in run_sync_team?
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Re-submitting #1866 again as requested:
cc @Kobzol
Discussed in #t-infra > removing inactive members from the org.