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

Add why3find dependency to nightly #1269

Merged
merged 2 commits into from
Nov 29, 2024
Merged

Add why3find dependency to nightly #1269

merged 2 commits into from
Nov 29, 2024

Conversation

Lysxia
Copy link
Collaborator

@Lysxia Lysxia commented Nov 29, 2024

D:

@Lysxia Lysxia enabled auto-merge November 29, 2024 09:12
@Lysxia Lysxia disabled auto-merge November 29, 2024 09:12
@Armael
Copy link
Contributor

Armael commented Nov 29, 2024

I think you also need to fix nightly.yml to pass --no-check-version for why3find

@jhjourdan
Copy link
Collaborator

I think you also need to fix nightly.yml to pass --no-check-version for why3find

Really? If the version does not match, then we need to know that, and update Why3find.

@Armael
Copy link
Contributor

Armael commented Nov 29, 2024

Well the point of the nightly job is to see whether creusot works with the newest (possibly different) version of why3find. (We already pass the same option for why3 right now.)

@jhjourdan
Copy link
Collaborator

I don't think the version check of why3find checks the individual commit.

@Lysxia
Copy link
Collaborator Author

Lysxia commented Nov 29, 2024

Nightly went green again

@Lysxia Lysxia merged commit 31d38fd into master Nov 29, 2024
5 checks passed
@Lysxia Lysxia deleted the fix-nightly branch November 29, 2024 09:49
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