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 coq-master CI #738

Merged
merged 1 commit into from
Feb 19, 2024
Merged

Fix coq-master CI #738

merged 1 commit into from
Feb 19, 2024

Conversation

rtetley
Copy link
Collaborator

@rtetley rtetley commented Feb 19, 2024

No description provided.

@rtetley rtetley requested a review from gares February 19, 2024 12:14
@rtetley
Copy link
Collaborator Author

rtetley commented Feb 19, 2024

I think merging #722 was a mistake as it was broken for coq-master branches.

@SkySkimmer
Copy link
Contributor

Isn't it just that the docker image is outdated?

@rtetley
Copy link
Collaborator Author

rtetley commented Feb 19, 2024

Well it was breaking on my machine as well for the nix builds which now seem to be fixed. However for the opam coq-dev set-up there seems to be a problem indeed.

@rtetley
Copy link
Collaborator Author

rtetley commented Feb 19, 2024

Actually you're probably right, it's the pin in the flake.nix that's outdated... Will fix in a minute

@rtetley rtetley merged commit 9b5230c into main Feb 19, 2024
15 checks passed
@rtetley rtetley deleted the fix-coq-master-ci branch March 21, 2024 09:24
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.

2 participants