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

Remove annoying warning for hover #28

Closed
zjhmale opened this issue Mar 24, 2017 · 8 comments
Closed

Remove annoying warning for hover #28

zjhmale opened this issue Mar 24, 2017 · 8 comments

Comments

@zjhmale
Copy link
Owner

zjhmale commented Mar 24, 2017

https://github.com/zjhmale/vscode-idris/blob/master/src/idris/commands.js#L41

@philipcraig
Copy link
Contributor

This message is indeed quite annoying. How can I install a pre-release, if the release is going to be a few days?

@zjhmale
Copy link
Owner Author

zjhmale commented Mar 26, 2017

@philipcraig, sorry for that, I will release it now 😅

@zjhmale
Copy link
Owner Author

zjhmale commented Mar 26, 2017

Hi @philipcraig, v0.3.7 is already released.

@philipcraig
Copy link
Contributor

philipcraig commented Mar 26, 2017

Thanks for the release! However, all I now get on all hovers is a message saying "didn't load path/to/source.idr" where the path is my path. Even hello world does this.

Edited: please disregard. I had accidentally done a stack install of a very old Idris version.

@zjhmale
Copy link
Owner Author

zjhmale commented Mar 26, 2017

@philipcraig That is because the Idris file itself can not be type checked, maybe I should show the error message directly instead of showing this confusing message.

@philipcraig
Copy link
Contributor

Good idea to show Idris's messages directly. If I'd been able to see Idris' message, which was about incompatible versions and advised cleaning the .ibc files, I'd have known what to do.

@zjhmale
Copy link
Owner Author

zjhmale commented Mar 26, 2017

Yes, we need to create an issue here, maybe you can do me the favor 😄 Thanks!

@philipcraig
Copy link
Contributor

will do

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

No branches or pull requests

2 participants