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

Compatible with Coq 8.20 #38

Merged
merged 1 commit into from
Mar 20, 2024
Merged

Compatible with Coq 8.20 #38

merged 1 commit into from
Mar 20, 2024

Conversation

pi8027
Copy link
Contributor

@pi8027 pi8027 commented Mar 20, 2024

No description provided.

@pi8027
Copy link
Contributor Author

pi8027 commented Mar 20, 2024

I haven't tested this locally. I'm letting CI do it.

@palmskog
Copy link
Member

@pi8027 is breakage for Coq 8.20 due to a recent change in Coq? I can see that CI passed for the mathcomp-dev Docker around 5 days ago.

@pi8027
Copy link
Contributor Author

pi8027 commented Mar 20, 2024

@palmskog It seems that coq/coq#18590 broke graph-theory.

@pi8027
Copy link
Contributor Author

pi8027 commented Mar 20, 2024

@palmskog
Copy link
Member

@pi8027 OK, but then maybe we should just drop 8.17 support directly? Or what is the plan for MathComp generally? I guess I go with whatever most MathComp projects do.

@pi8027
Copy link
Contributor Author

pi8027 commented Mar 20, 2024

I don't know "what most MathComp projects do" but MathComp itself already dropped 8.17 support.

@pi8027 pi8027 force-pushed the coq-8.20 branch 2 times, most recently from ed7ae39 to 0757a35 Compare March 20, 2024 14:46
@pi8027
Copy link
Contributor Author

pi8027 commented Mar 20, 2024

Done.

@proux01
Copy link
Contributor

proux01 commented Mar 20, 2024

Thanks, LGTM

@palmskog palmskog merged commit f047a60 into master Mar 20, 2024
5 checks passed
@palmskog palmskog deleted the coq-8.20 branch March 20, 2024 15:55
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