Skip to content

chore(ci): align toolchain to stable (fix cargo-fmt in CI)#141

Merged
lmlmt merged 2 commits into
hyperliquid-dex:masterfrom
kfastov:fix/align-toolchain-versions
Oct 20, 2025
Merged

chore(ci): align toolchain to stable (fix cargo-fmt in CI)#141
lmlmt merged 2 commits into
hyperliquid-dex:masterfrom
kfastov:fix/align-toolchain-versions

Conversation

@kfastov

@kfastov kfastov commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Why:

The Rust version is misaligned between CI and rust-toolchain.toml:

  • CI installs stable rustfmt/clippy;
  • rust-toolchain.toml requires nightly;

This causes cargo fmt to fail during CI.

Proposition:

  • We can safely change pinned toolchain version to "stable" because the repo doesn't use any nightly features yet

Changes:

  • Set rust-toolchain.toml to channel = "stable".
  • Fix clippy warnings to ensure CI passes

Result: build/fmt/clippy/tests run on stable; CI passes.

@kfastov kfastov marked this pull request as ready for review September 4, 2025 14:13
@kfastov

kfastov commented Sep 5, 2025

Copy link
Copy Markdown
Contributor Author

@lmlmt What do you think?

@lmlmt lmlmt self-requested a review October 20, 2025 23:27
@lmlmt

lmlmt commented Oct 20, 2025

Copy link
Copy Markdown
Contributor

@lmlmt What do you think?

Sounds good, thanks!

@lmlmt lmlmt merged commit 701ce47 into hyperliquid-dex:master Oct 20, 2025
1 check 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.

2 participants