Skip to content
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

chore: fix ci by hard-coding nightly version for miri #4825

Merged
merged 1 commit into from
Jul 12, 2022

Conversation

Noah-Kennedy
Copy link
Contributor

@Noah-Kennedy Noah-Kennedy commented Jul 12, 2022

It would seem that the latest nightly is having issues, so I went with the one from yesterday (2022-7-10).

It would seem that the latest nightly is having issues, so I went with the one from yesterday (2022-7-10).
@Noah-Kennedy Noah-Kennedy marked this pull request as ready for review July 12, 2022 01:50
@Noah-Kennedy Noah-Kennedy requested review from hawkw, carllerche and ipetkov and removed request for hawkw July 12, 2022 01:50
Copy link
Member

@ipetkov ipetkov left a comment

Choose a reason for hiding this comment

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

Nice thanks for fixing this!

@Noah-Kennedy Noah-Kennedy enabled auto-merge (squash) July 12, 2022 01:59
@Noah-Kennedy Noah-Kennedy merged commit be620e9 into master Jul 12, 2022
@Noah-Kennedy Noah-Kennedy deleted the noah/fix-miri branch July 12, 2022 02:14
Noah-Kennedy added a commit that referenced this pull request Jul 13, 2022
This reverts [#4825], which was to address Miri being broken on the latest nightly, possibly due to an issue in rustup.

Rustup issue: rust-lang/rustup#3031

[#4825]: #4825
@Darksonn Darksonn added the A-ci Area: The continuous integration setup label Jul 13, 2022
Noah-Kennedy added a commit that referenced this pull request Jul 13, 2022
This reverts [#4825], as miri seems to be working again on the latest nightly.

I had originally thought it was this issue with rustup which was to blame, but this seems to be wrong: rust-lang/rustup#3031

[#4825]: #4825
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-ci Area: The continuous integration setup
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants