Skip to content

fix(IDX): update publish-crates workflow#8693

Merged
cgundy merged 9 commits intomasterfrom
test-publish-crates
Feb 5, 2026
Merged

fix(IDX): update publish-crates workflow#8693
cgundy merged 9 commits intomasterfrom
test-publish-crates

Conversation

@cgundy
Copy link
Copy Markdown
Contributor

@cgundy cgundy commented Feb 5, 2026

This adds correct workflow permissions and introduces a dry-run mode.

Successful run here: https://github.com/dfinity/ic/actions/runs/21712164393/job/62618331297

Crate publishing based on this example: https://github.com/dfinity/ic-bn-lib/blob/main/.github/workflows/publish-ic-bn-lib.yml

@cgundy cgundy changed the title fix(IDX): update publish-crates workflow fix(IDX): update publish-crates workflow permissions Feb 5, 2026
@cgundy cgundy changed the title fix(IDX): update publish-crates workflow permissions fix(IDX): update publish-crates workflow Feb 5, 2026
@cgundy cgundy marked this pull request as ready for review February 5, 2026 12:53
@cgundy cgundy requested review from a team as code owners February 5, 2026 12:53
Copy link
Copy Markdown
Member

@jwndlng jwndlng left a comment

Choose a reason for hiding this comment

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

LGTM! As a minor nit: we could be more explicit by checking == 'false' in the second if.

@cgundy cgundy added this pull request to the merge queue Feb 5, 2026
Merged via the queue into master with commit c062791 Feb 5, 2026
39 checks passed
@cgundy cgundy deleted the test-publish-crates branch February 5, 2026 14:29
kpop-dfinity pushed a commit that referenced this pull request Feb 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants