Skip to content

Conversation

@Kobzol
Copy link
Member

@Kobzol Kobzol commented Nov 29, 2025

@rustbot
Copy link

rustbot commented Nov 29, 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

@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

@jieyouxu jieyouxu added needs-team-repo-admin-review This change requires one of the `team-repo-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. needs-infra-admin-review This change requires one of the `infra-admins` to review. and removed needs-team-repo-admin-review This change requires one of the `team-repo-admins` to review. labels Dec 1, 2025
Copy link
Member

@Mark-Simulacrum Mark-Simulacrum left a comment

Choose a reason for hiding this comment

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

I think once we have support for this we will also need to add support to sync owners, permitting adding only teams (individual owners can remove rust-lang-owner so shouldn't be allowed).

@marcoieni
Copy link
Member

The dry run is clean: #2138 (comment)

@marcoieni marcoieni enabled auto-merge December 2, 2025 07:51
@marcoieni marcoieni disabled auto-merge December 2, 2025 07:51
@Kobzol Kobzol force-pushed the trusted-publishing-only branch from 95447ae to cf3693b Compare December 2, 2025 07:56
@marcoieni marcoieni enabled auto-merge December 2, 2025 07:57
@marcoieni marcoieni added this pull request to the merge queue Dec 2, 2025
Merged via the queue into rust-lang:main with commit 4bf6fe1 Dec 2, 2025
3 checks passed
@Kobzol Kobzol deleted the trusted-publishing-only branch December 2, 2025 08:16
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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants