Skip to content

Use new toolchain file #281

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

Closed
wants to merge 2 commits into from
Closed

Conversation

DJMcNab
Copy link
Contributor

@DJMcNab DJMcNab commented Nov 27, 2020

I'm not quite sure where is best to document that we require nightly rust, but in that list of instructions is definitely not it.

@DJMcNab
Copy link
Contributor Author

DJMcNab commented Nov 28, 2020

Realistically, blocked on rust-lang/rustup#2579.

@DJMcNab DJMcNab closed this Nov 28, 2020
@XAMPPRocky
Copy link
Member

I don't see how that blocks this? I think removing the setup scripts and automatically installing components is still valuable even if it installs an extra component on CI, rustup is not where we're spending most of our time in CI anyway.

@DJMcNab
Copy link
Contributor Author

DJMcNab commented Nov 30, 2020

Yeah, it also seems like the GitHub actions rustup was not up to date, and I don't have time to work out how to work around that at the moment

Also I did find a workaround in that thread, so we should use that

I might reopen this in the next few days, but it might be worth whoever does the next rust update just doing it anyway

@DJMcNab DJMcNab deleted the toolchain branch December 12, 2020 12:44
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