-
Notifications
You must be signed in to change notification settings - Fork 2
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
🔣 A domain-specific language with handler combinators #186
Comments
I was going to write a response here that something like a handler combinator thingy is the more natural way to proceed, but you beat me to it :) |
@kentookura Are you claiming that with these three combinators you can already implement the features you wanted trivially? |
Yes, I think so |
One thing that I want to point out is that when implementing a language server, it is important that an empty list of diagnostics gets pushed when a computation that previously failed now succeeded. Right now I am checking this manually, but maybe this is an interesting factor when designing this. Another thing is that diagnostics need a URI to be published to, and a computation might raise diagnostics for many files. I am handling this by
This is not enough, though: |
refactors: - create modules for handlers to make `Lsp_server` file more manageable. - store import graph in state - pull out the code that publishes diagnostics into separate module to prepare for smarter diagnostics reporting: RedPRL/asai#186 issues: - When running the evaluator, all errors get reported in the currently opened file. Need to accumulate and report separately, see the above github issue.
The text was updated successfully, but these errors were encountered: