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

adapt to MC#1229 #58

Merged
merged 1 commit into from
Aug 14, 2024
Merged

adapt to MC#1229 #58

merged 1 commit into from
Aug 14, 2024

Conversation

Tragicus
Copy link
Contributor

@Tragicus Tragicus commented Aug 2, 2024

Some proofs rely on the fact that the proof of membership in I_n comes from inZp, which is not the case anymore with math-comp/math-comp#1229.

@Tragicus
Copy link
Contributor Author

Tragicus commented Aug 8, 2024

@CohenCyril I tried and failed to figure out why the CI does not find coq-elpi. Is it fine anyway? Otherwise, do you know what happens?

@CohenCyril
Copy link
Member

@CohenCyril I tried and failed to figure out why the CI does not find coq-elpi. Is it fine anyway? Otherwise, do you know what happens?

I am rather puzzled, I cannot reproduce this error on my computer. Will take another look tomorrow.

@Tragicus Tragicus force-pushed the pr1229 branch 2 times, most recently from a449871 to ef2bfc0 Compare August 14, 2024 08:13
@CohenCyril CohenCyril merged commit b21f200 into math-comp:master Aug 14, 2024
20 checks passed
@Tragicus Tragicus deleted the pr1229 branch August 14, 2024 14:25
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