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

fix: Partially fix nightly build for 4.2 #131

Merged
merged 4 commits into from
Jul 11, 2023

Conversation

robin-aws
Copy link
Member

Only fixes the fact that the top-level workflow looks green even though the job fails (https://github.com/dafny-lang/libraries/actions/runs/5506948866) and adds --compile-suffix when necessary.

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

Suspected as the reason nightly build looks green when it actually isn’t
@robin-aws robin-aws changed the title fix: Fix nightly build for 4.2 fix: Partially fix nightly build for 4.2 Jul 10, 2023
atomb
atomb previously approved these changes Jul 11, 2023
Copy link
Member

@atomb atomb left a comment

Choose a reason for hiding this comment

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

I'm on board with merging this even though the 4.1 tests fail, to make sure that our next PR has to be one to fix those tests. However, do the settings on this repository allow us to merge a PR with failing tests?

@robin-aws
Copy link
Member Author

I'm on board with merging this even though the 4.1 tests fail, to make sure that our next PR has to be one to fix those tests. However, do the settings on this repository allow us to merge a PR with failing tests?

When I cut this PR I thought I was just making progress on working against the latest nightly, but now I see that 4.1 has never worked. :(

I'll merge this by temporarily changing the branch protection rules, just because I think turning the nightly build red as it should be is important for visibility as well.

Copy link
Member

@MikaelMayer MikaelMayer left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks!

Copy link
Member

@atomb atomb left a comment

Choose a reason for hiding this comment

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

Let's see if we can get things working with 4.1 soon, but this is still better than it was. At least every PR gets the notification that it's still broken.

@robin-aws robin-aws merged commit 4c4fb06 into dafny-lang:master Jul 11, 2023
5 of 6 checks passed
@robin-aws
Copy link
Member Author

I've re-added 4.1 as a required check, so the CI is effectively broken on this repo now as intended. :)

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.

3 participants