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

move notations into module #233

Closed
wants to merge 1 commit into from

Conversation

JoJoDeveloping
Copy link
Contributor

This notation is reserved and only used in some parts of the development. But since it is still globally reserved, importing vec.v still produces warnings about clashing notations. A proper fix is to move these notations into a module.

This is a first draft, presumably the definition should also be moved into such a module so that one does not have to re-define the notation elsewhere.

Making the notation no longer globally enabled would be great because it otherwise clashes with the ones defined in FOL/Syntax/SyntacticOps.v.

@DmxLarchey
Copy link
Collaborator

@JoJoDeveloping Please have a look at PR #229 that does precisely that among other things. It was only merged in master though.

@JoJoDeveloping
Copy link
Contributor Author

Oh, thanks for the hint. That seems to do the trick, sad that it's not available on 8.20 yet. So I guess we just wait until you're updated to 9.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