Skip to content

Use trusted publishing for chalk - #2806

Merged
marcoieni merged 2 commits into
rust-lang:mainfrom
Kobzol:chalk-crates
Oct 12, 2026
Merged

marcoieni merged 2 commits into
rust-lang:mainfrom
Kobzol:chalk-crates

Conversation

@Kobzol

@Kobzol Kobzol commented Oct 2, 2026 •

Copy link
Copy Markdown
Member

Now that we have the chalk crates under rust-lang-owner, we can manage them in team.

Companion PR: rust-lang/chalk#835

@github-actions

github-actions Bot commented Oct 2, 2026 •

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 `chalk-derive` to `true`
      Setting trusted publishing only option for krate `chalk-engine` to `true`
      Setting trusted publishing only option for krate `chalk-ir` to `true`
      Setting trusted publishing only option for krate `chalk-macros` to `true`
      Removing `user` owner `nikomatsakis` from krate `chalk-macros`
      Removing `team` owner `github:rust-lang:core` from krate `chalk-macros`
      Removing `team` owner `github:rust-lang:compiler` from krate `chalk-macros`
      Removing `team` owner `github:rust-lang:wg-traits` from krate `chalk-macros`
      Setting trusted publishing only option for krate `chalk-recursive` to `true`
      Setting trusted publishing only option for krate `chalk-rust-ir` to `true`
      Removing `user` owner `nikomatsakis` from krate `chalk-rust-ir`
      Removing `user` owner `jackh726` from krate `chalk-rust-ir`
      Removing `team` owner `github:rust-lang:wg-traits` from krate `chalk-rust-ir`
      Setting trusted publishing only option for krate `chalk-solve` to `true`

[INFO rust_team::sync] synchronizing github
[INFO rust_team::sync] 💻 Repo Diffs:
📝 Editing repo 'rust-lang/chalk':
Environments:
➕ Create: publish
Branches: master

@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 Oct 2, 2026
Comment thread repos/rust-lang/chalk.toml Outdated
"chalk-derive",
"chalk-engine",
"chalk-ir",
"chalk-macros",

@marcoieni marcoieni Oct 12, 2026 •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

should we move chalk-macros to the obsolete crate section?
this crate is not part of the workspace anymore.
It was renamed in 2020 rust-lang/chalk@896f786

(found with AI)

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. FWIW, I think that all the crates are obsolete, but since they had CI publishing set up, and chalk wasn't yet archived, I wanted to keep it working at least somewhat.

@marcoieni
marcoieni added this pull request to the merge queue Oct 12, 2026
Merged via the queue into rust-lang:main with commit d8d6e00 Oct 12, 2026
5 checks passed
@marcoieni

Copy link
Copy Markdown
Member

I also deleted the token from github actions secrets revoked it from crates-io image

@Kobzol
Kobzol deleted the chalk-crates branch October 12, 2026 18:02
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.

3 participants