Skip to content

Revert "Feat rust operators (#5380)" (again)#5383

Merged
robin-aws merged 2 commits into
dafny-lang:masterfrom
robin-aws:revert-feat-rust-operators-again
May 1, 2024
Merged

Revert "Feat rust operators (#5380)" (again)#5383
robin-aws merged 2 commits into
dafny-lang:masterfrom
robin-aws:revert-feat-rust-operators-again

Conversation

@robin-aws

@robin-aws robin-aws commented Apr 30, 2024

Copy link
Copy Markdown
Member

This reverts commit 0b629ea to fix failing nightly: https://github.com/dafny-lang/dafny/actions/runs/8896634004/job/24436389955

Unfortunately I approved #5380 before all of the deep tests had run, and because the extra checks aren't normally run for a PR unless it has the run-deep-tests label, they can't be required checks, so the PR was auto-merged even though several Windows runs timed out after 2 hours.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT license.

@robin-aws robin-aws changed the title Revert "Feat rust operators (#5380)" Revert "Feat rust operators (#5380)" (again) Apr 30, 2024
@robin-aws robin-aws added the run-deep-tests Tells CI to run all tests label Apr 30, 2024
@robin-aws robin-aws merged commit 88808ee into dafny-lang:master May 1, 2024
@robin-aws robin-aws deleted the revert-feat-rust-operators-again branch May 1, 2024 03:10
@robin-aws

Copy link
Copy Markdown
Member Author

Temporarily allowed administrators to merge without checks passing, because the osx unit tests were hitting flaky failures over and over, and this is a revert intended to unblock nightly anyway.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-deep-tests Tells CI to run all tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants