Skip to content

Conversation

@Sh3Rm4n
Copy link
Collaborator

@Sh3Rm4n Sh3Rm4n commented Oct 24, 2025

No description provided.

@Sh3Rm4n Sh3Rm4n self-assigned this Oct 24, 2025
@Sh3Rm4n Sh3Rm4n force-pushed the ci/use-rust-toolchain branch from 6e8fd10 to b6b9138 Compare October 24, 2025 08:02
@Sh3Rm4n Sh3Rm4n requested a review from cathay4t October 24, 2025 08:13
@cathay4t
Copy link
Member

cathay4t commented Nov 30, 2025

This action is not widely used yet and current step is simple enough to maintain.

I have history on using github action to install rust, then that action got deprecated. So instead of hoping others maintains that action, I prefer to do simple task by myself.

No issue no change.

@Sh3Rm4n
Copy link
Collaborator Author

Sh3Rm4n commented Dec 2, 2025

I understand, that you don't want to use external external dependencies. What do you think about the rust-toolchain.toml file? I could revert the CI changes and keep that. WDYT?

@@ -0,0 +1,3 @@
[toolchain]
channel = "1.77.0"
Copy link
Member

Choose a reason for hiding this comment

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

This file is forcing people to install 1.77 when running cargo build. That's why I don't use this method but use CI to check MSRV.

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