Skip to content

configure trusted publishing for libc - #2619

Merged
ubiratansoares merged 1 commit into
mainfrom
configure-trusted-publishing-for-libc
Aug 3, 2026
Merged

configure trusted publishing for libc#2619
ubiratansoares merged 1 commit into
mainfrom
configure-trusted-publishing-for-libc

Conversation

@marcoieni

@marcoieni marcoieni commented Jul 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Related to rust-lang/libc#5317
When merging this, rust-lang-owner should remove individual access and undeclared teams

@marcoieni
marcoieni force-pushed the configure-trusted-publishing-for-libc branch 2 times, most recently from e11dc11 to 237abff Compare July 22, 2026 17:08
@marcoieni marcoieni mentioned this pull request Jul 22, 2026
Merged
5 tasks
@marcoieni
marcoieni marked this pull request as ready for review July 22, 2026 17:11
@marcoieni marcoieni mentioned this pull request Jul 22, 2026
Open
28 tasks
@github-actions

github-actions Bot commented Jul 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Dry-run check results 

[WARN  rust_team::sync] sync-team is running in dry mode, no changes will be applied.
[INFO  rust_team::sync] synchronizing crates-io
[INFO  rust_team::sync] 💻 Trusted Publishing Crate Diffs:
      Setting trusted publishing only option for krate `ctest` to `true`
      Adding `team` owner `github:rust-lang:libs` to krate `ctest`
      Setting trusted publishing only option for krate `libc` to `true`
      Removing `team` owner `github:rust-lang:libc` from krate `libc`

[INFO  rust_team::sync] synchronizing github

[INFO  rust_team::sync] 💻 Repo Diffs:

📝 Editing repo 'rust-lang/libc':

Environments:

➕ Create: release

Branches: main, libc-0.2

tgross35
tgross35 reviewed Jul 22, 2026
View reviewed changes
Comment thread repos/rust-lang/libc.toml Outdated
Comment on lines +40 to +41
[environments.release]
branches = ["main"]

@tgross35 tgross35 Jul 22, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We need libc-0.2 as well

@marcoieni marcoieni Jul 22, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

done, if you want to generalize in the future, you can have libc-* both in the ruleset and in the environment I think

@marcoieni
marcoieni force-pushed the configure-trusted-publishing-for-libc branch from 237abff to b1156d3 Compare July 22, 2026 17:21
tgross35
tgross35 approved these changes Jul 22, 2026
View reviewed changes

@tgross35 tgross35 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@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. labels Jul 26, 2026
@tgross35

tgross35 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Can this be merged or is something else needed?

@ubiratansoares
ubiratansoares added this pull request to the merge queue Aug 3, 2026
@ubiratansoares

ubiratansoares commented Aug 3, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

@tgross35 Done!

I'll delete tokens after rust-lang/libc#5317 merged!

Merged via the queue into main with commit 1a19e99 Aug 3, 2026
4 checks passed
@ubiratansoares
ubiratansoares deleted the configure-trusted-publishing-for-libc branch August 3, 2026 19:15
@tgross35

tgross35 commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Thanks! In the queue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ubiratansoares ubiratansoares ubiratansoares approved these changes
+1 more reviewer
@tgross35 tgross35 tgross35 approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@ubiratansoares ubiratansoares

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@marcoieni @tgross35 @ubiratansoares @jieyouxu