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

Update CI #104

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Update CI #104

merged 1 commit into from
Dec 6, 2024

Conversation

pi8027
Copy link
Member

@pi8027 pi8027 commented Dec 5, 2024

No description provided.

@pi8027
Copy link
Member Author

pi8027 commented Dec 5, 2024

@gares Hi, can the following error be turned into a warning, so that we can keep compatibility with older versions of Coq?

  # File "./theories/lra.v", line 397, characters 0-15:
  # Error: The command 'Elpi Typecheck' is deprecated (and does nothing) since
  # type checking is now performed by 'Elpi Accumulate'.
  # [elpi.typecheck-syntax,elpi.deprecated,deprecated,elpi,default]

@pi8027
Copy link
Member Author

pi8027 commented Dec 5, 2024

Or is it broken because of -arg -w -arg +elpi.typecheck in Make?

@gares
Copy link
Member

gares commented Dec 5, 2024

Add a -typecheck-syntax, I think I did that in HB

@pi8027
Copy link
Member Author

pi8027 commented Dec 6, 2024

I still don't get why it is not a warning but an error by default, but thanks anyway.

@pi8027 pi8027 merged commit 16b3dbf into master Dec 6, 2024
20 checks passed
@pi8027 pi8027 deleted the ci branch December 6, 2024 12:49
@gares
Copy link
Member

gares commented Dec 6, 2024

I changed the default in 2.3.0

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