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

CI: update for Coq 8.19+rc1 #718

Merged
merged 2 commits into from
Jan 2, 2024
Merged

Conversation

hendriktews
Copy link
Collaborator

  • add tests with 8.19+rc1
  • retire some 8.15 tests, because 8.15 is now older than 18 month

File ci/test_wholefile.v file is outdated, uses deprecated features,
is therefore prone to caues Coq errors and therefore causes the tests
that use this file to fail. See ProofGeneral#698 and commit
bd3615b.
- add tests with 8.19+rc1
- retire some 8.15 tests, because 8.15 is now older than 18 month
@hendriktews
Copy link
Collaborator Author

Hi Erik and Pierre,
this PR adds Coq 8.19-rc1 to CI. I disabled 060_coq-test-wholefile and 070_coq-test-regression-wholefile-no-proof - for the third time in a row, test_wholefile.v made problems, because it is not maintained. I suggest to include these tests only when somebody starts maintaining this file.
If you don't object, I'll merge this PR next week.
Happy New Year!
Hendrik

Copy link
Member

@erikmd erikmd left a comment

Choose a reason for hiding this comment

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

Thanks Hendrik! LGTM

Best wishes ✨️

@hendriktews hendriktews merged commit a6017ae into ProofGeneral:master Jan 2, 2024
119 checks passed
@hendriktews hendriktews deleted the ci-19-rc branch January 6, 2024 20:51
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