Skip to content

Conversation

@alexmohr
Copy link
Contributor

Summary

some more minor tweaks for publishing to crates.io

  • add taplo check
  • add release stage
  • add versions to Cargo.toml, to enable publishing on crates.io

This is a draft because I'm still in process of clarifying how we get the necessary token.
The publish stage is taken from https://github.com/eclipse-uprotocol/up-rust/blob/main/.github/workflows/release.yaml#L184

Checklist

  • I have tested my changes locally
  • I have added or updated documentation
  • I have linked related issues or discussions
  • I have added or updated tests

Related

Notes for Reviewers

this way cargo publish is using the version
and for local development, we still can refer to the workspace

Signed-off-by: Alexander Mohr <[email protected]>
Signed-off-by: Alexander Mohr <[email protected]>
with:
submodules: false
- name: Install Rust toolchain
uses: dtolnay/rust-toolchain@nightly

Choose a reason for hiding this comment

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

question: do we need nightly toolchain for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed, using only 1.88.0 as defined in the parameters

Signed-off-by: Alexander Mohr <[email protected]>
@alexmohr alexmohr marked this pull request as ready for review December 1, 2025 10:35
@alexmohr alexmohr requested a review from a team as a code owner December 1, 2025 10:35
Copy link

@theswiftfox theswiftfox left a comment

Choose a reason for hiding this comment

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

LGTM :)

@alexmohr alexmohr merged commit cc5e83e into main Dec 1, 2025
11 checks passed
@alexmohr alexmohr deleted the bugfix/publish branch December 1, 2025 14:04
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