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

Don't test on nightly #4453

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Don't test on nightly #4453

merged 1 commit into from
Feb 29, 2024

Conversation

dureuill
Copy link
Contributor

Pull Request

Related issue

Fixes #4441 better 😅

What does this PR do?

  • No longer run tests on nightly

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.

@dureuill dureuill added maintenance Issue about maintenance (CI, tests, refacto...) skip changelog The PR will not appear in the release changelogs labels Feb 29, 2024
@dureuill dureuill added this to the v1.7.0 milestone Feb 29, 2024
@dureuill dureuill requested a review from a team February 29, 2024 13:28
Copy link
Member

@curquiza curquiza left a 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?

@dureuill
Copy link
Contributor Author

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

Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

LGTM ✅

Copy link
Member

@irevoire irevoire left a 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 😔

@curquiza
Copy link
Member

curquiza commented Feb 29, 2024

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 😅

@dureuill
Copy link
Contributor Author

bors merge

I guess 😅

Copy link
Contributor

meili-bors bot commented Feb 29, 2024

@dureuill dureuill merged commit f4a6261 into release-v1.7.0 Feb 29, 2024
10 checks passed
@dureuill dureuill deleted the remove_nightly_tests branch February 29, 2024 15:23
@curquiza curquiza removed the skip changelog The PR will not appear in the release changelogs label Feb 29, 2024
@meili-bot meili-bot added the v1.7.0 PRs/issues solved in v1.7.0 released on 2024-03-11 label Mar 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Issue about maintenance (CI, tests, refacto...) v1.7.0 PRs/issues solved in v1.7.0 released on 2024-03-11
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants