Fork me on GitHub
#lsp
<
2022-01-29
>
lassemaatta06:01:08

I noticed recently that clojure-lsp spends some time after startup analyzing something test related ("Refreshing testTree took 46.10 secs" etc). Is this something related to lsp-clojure-show-test-tree?

lassemaatta06:01:26

and is this something that could be perhaps disabled with some configuration (assuming disabling it doesn't cripple anything critical)?

lassemaatta07:01:40

my motivation here is that loading a project already takes a considerable amount of time and if I can disable (or postpone) any expensive non-critical analysis with some config is great

ericdallo13:01:15

Yes, and it's async

ericdallo13:01:14

I think there is no flag to disable that, but I'd be possible, but I don't think it waste too mamy resources, maybe you can profile or something

lassemaatta13:01:50

Sure, this isn’t a major issue as it’s async, so no worries

👍 1
jacob.maine23:01:54

make test is failing on master. java.lang.ClassNotFoundException: clojure_lsp.feature.test_tree.TestTreeParams

jacob.maine23:01:52

In the cli tests. lib tests are ok

ericdallo23:01:59

I think I fixed with latest commit (some minutes ago), right?

ericdallo23:01:48

Actually that commit, was the fix 😅

ericdallo23:01:56

You can check by the CI

jacob.maine23:01:13

Ah, yes, you’re right. Thanks!

👍 1
wcohen16:02:25

@UKFSJSM38 fighting with CI is always how you want to start your day, right?

ericdallo16:02:48

Not really... Man, I hate Windows with all my forces 😂

ericdallo16:02:05

But this time is related with the tools.build migration

ericdallo16:02:33

I should have spent more time testing that, but I found the issue I think, 🤞

wcohen17:02:16

@UKFSJSM38 new release seems good for me, happy to submit a nixpkgs pr if useful

ericdallo17:02:46

thanks! that'd help a lot, you will probably just need to make some changes removing some datalevin code there

wcohen17:02:23

just the broken note, right?

ericdallo17:02:21

I think so, actually, feel free to remove all that code and just call the new make native-cli

ericdallo17:02:30

it should make the derivation even easier

wcohen17:02:57

looks like the buildPhase has already been abstracted out, except for the preBuild stuff that had to get added in for 01.22

ericdallo17:02:15

yes, I forgot about that