Fix and improve "type at cursor" command; show errors in phantoms#20
Fix and improve "type at cursor" command; show errors in phantoms#20bluepichu wants to merge 20 commits into
Conversation
Types and errors in phantoms
Signature support and better verbosity handling
|
I didn't realize it, but this would close #19 I suppose. |
|
That looks very cool! I am tempted to switch to st3 :) |
|
Hey @bluepichu, thank you for taking time to contribute to this project! Let me know when I can start reviewing your changes. 🙌 |
|
@cynddl - you can review this whenever you're ready :) |
|
@bluepichu Great work! I made a few cosmetic changes and one fix (3bf022d):
I feel like the main file is getting bigger and bigger and would appreciate if we can separate display functionalities (eg. showing phantoms) from commands and event listeners. Maybe by creating a merlin/phantom.py file with all the display-related code. Let me know if you can do that, otherwise I'll see if I have time this week. Thanks again! 🎉 |
|
Hi @cynddl -- looks great! I definitely won't have time for the remainder of this week, so definitely don't wait on me for any restructuring you want to do. |
Don't merge this, it's not ready yet! I just wanted to make sure it was on your radar before it's done.
Hi there! I found this package in my preparation to do a bunch of OCaml dev in Sublime. I'm glad it's here :)
However, I found that the "type at cursor" text command didn't seem to work, and the related context menu command was kind of lacking. This PR should provide a better interface for accessing types. While I was doing this, I also moved errors into phantoms to get rid of bottom pop-up altogether.
Here are my changes in action on a sample
catprogram:Important caveat: this DOES NOT work properly in the current "stable" version of ST3, since phantoms have a major bug that sometimes causes them to display the incorrect content. However, it does work properly on the latest dev build.UPDATE: A new stable ST3 just got released that fixed this bug.Remaining to-do