emacs-fsharp-mode
emacs-fsharp-mode copied to clipboard
Linting and Colorization
I noticed that linting and colorization are supported by fsautocomplete. Do you want PRs for these?
That would be great :-).
- Linting. I would probably leave this off by default so as not to surprise people.
- Colorization. It is only the very basic extended information about new computation expressions, as far as I know. You can see what it gives in the corresponding FSAC test. I think the easiest place to hook it in, if you think it is worthwhile, would be to make this function aware of some variable https://github.com/fsharp/emacs-fsharp-mode/blob/master/fsharp-mode-font.el#L216. You can see from the string case that you can do arbitrary processing, so checking a pre-populated variable should be easy.
For linting, I was thinking it might be a good idea to integrate it with flycheck.
Could do the same for compiler errors too.
For linting, I was thinking it might be a good idea to integrate it with flycheck.
I'm not intrinsically against it, but what are the benefits compared to the current approach?
The error list can be useful and it has plugins for most languages. It's very widely used.
Also, using that would probably cut down the amount of code in emacs-fsharp-mode.
Also, using that would probably cut down the amount of code in emacs-fsharp-mode.
That's reason enough :-)
Sure I would accept a PR on that basis.
If possible flycheck is definitely the way to go here.
Just wanted to say that I've not forgotten about this. I got linting working pretty quickly but hit a bug where it was linting continuously (even while not typing) and just not had chance to go back and fix it yet.
![screen shot 2016-02-09 at 14 14 31](https://cloud.githubusercontent.com/assets/667194/13880146/d9a3fffa-ed12-11e5-8e63-bcb65391020b.png)
Cool that's looking really nice!
@nosami Which theme is this?
@delexi it's gruvbox
:+1: for flycheck
@nosami Do you have an up2date branch of your work I can try?
@juergenhoetzel Not up to date, but it's here https://github.com/nosami/emacs-fsharp-mode. Sorry, not had any time!
Thanks. I will give it a try.
Since flycheck support is implemented, I suspect this issue can be closed :question:
We don't have colorization, but it isn't a big deal.