-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[gh-1554] upgrade rust version and fix syntax errors. #1555
[gh-1554] upgrade rust version and fix syntax errors. #1555
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
f9043f5
to
312d66c
Compare
Help debug GitHub CI. @jolestar @popcnt1 @yourmoonlight |
It seems like Rustup does not support it.
|
Will work on it soon. @jolestar Let me figure it out, take a break |
@feliciss try to remove inputs in .github/actions/rust-setup/action.yaml in action-rs(https://github.com/marketplace/actions/rust-toolchain) reason: CI rely on actions-rs/toolchain to install rust toolchain, and " Input has higher priority, so if you are want to use toolchain file, you need to remove the input from the workflow file." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- default profile to install toolchain
- fix input in rust-setup/action.yaml
- try 1.75.0? (it's a good time to upgrade rust)
312d66c
to
a9e2cbb
Compare
I tried to remove action.yaml but there still has issue. |
|
e646c68
to
2f1b2dd
Compare
I made this PR to just upgrading rust version. The rust-toolchain file format remains the same to be resolved. |
99232e9
to
6aaab85
Compare
Summary
Upgrade rust version and fix errors.