flutter.el icon indicating copy to clipboard operation
flutter.el copied to clipboard

Run tests on lsp-dart

Open ericdallo opened this issue 5 years ago • 2 comments

I think I fixed our regex problems in lsp-dart :) Sorry for noticing only now that VSCode uses LSP to build the tests lens and do not need that crazy regex.

Maybe we can remove the tests functions if you want it

ericdallo avatar Apr 18 '20 21:04 ericdallo

Thanks for letting me know. It's probably unlikely, but just in case there's anyone who wants to use flutter.el and not lsp-dart (maybe people who prefer eglot?) I'll keep them for now. I do like the easy bindings we have via the minor mode.

Incidentally I've added lsp-dart but I can't get those lenses to show up. Is there something special I have to do?

amake avatar Apr 19 '20 13:04 amake

It makes sense, thanks!

Yeah, the dart_analysis_server from dart sdk just released this for LSP on version 2.8, so you need dart 2.8 or above to test this features :/ I think I mentioned this requirement on README but for other features, this is missing, I'll add it, thanks!

ericdallo avatar Apr 19 '20 14:04 ericdallo