Skip to content

Don't use force in cargo publish. - #100

Merged
dmkozh merged 2 commits into
stellar:mainfrom
dmkozh:publish_no_force
Apr 7, 2026
Merged

Don't use force in cargo publish.#100
dmkozh merged 2 commits into
stellar:mainfrom
dmkozh:publish_no_force

Conversation

@dmkozh

@dmkozh dmkozh commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

It's not clear why it was there in the first place, but we do know now that it prevents us from restarting the publishes that failed after some crates have been published (e.g. due to spurious crates io issues). Due to the force flag we try to re-publish an already existing crate, which is not allowed and results in the publish process abort.

It's not clear why it was there in the first place, but we do know now that it prevents us from restarting the publishes that failed after some crates have been published (e.g. due to spurious crates io issues). Due to the `force` flag we try to re-publish an already existing crate, which is not allowed and results in the publish process abort.
Copilot AI review requested due to automatic review settings April 6, 2026 18:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates the Rust publishing GitHub Actions workflow to avoid forcing re-publishes, enabling safe re-runs when a previous publish partially succeeded (e.g., crates.io transient failures).

Changes:

  • Remove --force '*' from cargo workspaces publish invocation to prevent attempts to re-publish already-published crates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dmkozh
dmkozh enabled auto-merge (squash) April 7, 2026 15:38
@dmkozh
dmkozh merged commit 3f06bc8 into stellar:main Apr 7, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants