-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Don't test on nightly #4453
Don't test on nightly #4453
Conversation
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.
Are we ok with nightly for RustFmt job?
yes I think we are. Historically there has been very low breakage due to rustfmt, and also we're actively using unstable options for formatting |
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.
LGTM ✅
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.
Yep, we don’t have a choice, rustfmt doesn’t work on stable with our options 😔
I feel like I always ask the question and I remember I already asked you the same question months ago, but I only remember when I see your answer again 😅 |
bors merge I guess 😅 |
Pull Request
Related issue
Fixes #4441 better 😅
What does this PR do?
The motivation for this change is that we are now updating Rust at fixed points in time, and so no longer need nightly runs to ensure that a change won't get into stable and break our build at the worst possible moment.