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

Fix #46 #67

Open
wants to merge 2 commits into
base: pathogen-bundle
Choose a base branch
from

Conversation

dwarfmaster
Copy link
Contributor

This is the continuation of my previous pull request #55 : it updates the way coquille handles Coq error messages to prevent python to fail on those messages.

Python no longer fails for most of coq errors. When sending an invalid
argument during a proof, an error is displayed in the info box, but the
cursor moves over the instruction : it may not be intuitive.
Furthermote, moving to a Qed like that creates another python error.
@neshkeev
Copy link

neshkeev commented Nov 6, 2017

It works for me. It would be great to see the commits in the main repository.

@fangyi-zhou
Copy link

fangyi-zhou commented Nov 13, 2017

I'm receiving parse xml errors in vim
(random xml printed to terminal)

@uuner
Copy link

uuner commented May 17, 2018

Please, integrate this change. Coquille does not work without it and works with it.
Coq version 8.7.2
VIM version 8.0

@tekknolagi
Copy link

Also checking in that Coquille breaks without this.

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.

5 participants