Skip to content

Conversation

@rustbot
Copy link

rustbot commented Nov 11, 2025

rust_team_data/src/v1.rs has been modified, it is used (as a git dependency) by multiple sub-projects like triagebot, the www.rust-lang.org website and others.

If you are changing the data structures, please make sure that the changes are not going to break serde deserialization (adding a field is fine; removing or renaming a field isn't).

If you must do a breaking change to the format, make sure to coordinate it with all the users of the rust_team_data crate.

cc @Urgau

@jieyouxu jieyouxu added needs-infra-admin-review This change requires one of the `infra-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. T-infra Relevant to the infrastructure team. labels Nov 11, 2025
marcoieni
marcoieni previously approved these changes Nov 11, 2025
@marcoieni
Copy link
Member

oops, just noticed CI fails

@Kobzol
Copy link
Member Author

Kobzol commented Nov 11, 2025

Fixed CI (hopefully).

@github-actions
Copy link

Dry-run check results

[WARN  sync_team] sync-team is running in dry mode, no changes will be applied.
[INFO  sync_team] synchronizing github

@marcoieni marcoieni added this pull request to the merge queue Nov 11, 2025
Merged via the queue into rust-lang:main with commit a7b38de Nov 11, 2025
3 checks passed
@Kobzol Kobzol deleted the github-sponsors branch November 11, 2025 18:04
@lcnr lcnr mentioned this pull request Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-infra-admin-review This change requires one of the `infra-admins` to review. S-waiting-on-review Status: waiting on review from a team/WG/PG lead, an infra-admin, and/or a team-repo-admin. T-infra Relevant to the infrastructure team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants