cursive

Issues at: https://git.io/cursive-issues
imre 2021-04-22T05:30:56.237700Z

One way perhaps could be that paths that are in aliases/test/extra-paths and also under aliases/somethingelse/extra-paths could be handled per point 2. Approaching another way: only paths that only appear in the test alias would be automarked as test source roots

imre 2021-04-22T05:31:08.237900Z

Of course this could break another edge case

2021-04-22T06:29:03.238100Z

Makes sense to only mark paths that are exclusively in :test alias as test source. Test classpath needs, by nature, to include source under test, and tools-deps doesnโ€™t provide a way to inherit paths from other aliases, only from the top level :paths. If I include src/clj in the top level :paths it will also be part of the ClojureScript part of our build, which I am trying to avoid. Of course there is the possibility to invoke test with more aliases to combine the needed classpath, something like clj -A:clj-src:test , but there is something nice about expecting all projects to run tests with the simple clj -A:test

borkdude 2021-04-22T10:46:08.239400Z

Are there any clj-kondo users that can help @jtlocsei?

kari 2021-04-23T07:16:23.240900Z

Heh, I am happy I was able to help anyway. ๐Ÿ˜€

borkdude 2021-04-23T07:33:16.241100Z

@kari.marttila Take care!

kari 2021-04-23T07:42:10.241300Z

I will! Thanks!

borkdude 2021-04-22T11:15:07.239700Z

He just let me know that it works using the LSP plugin

kari 2021-04-22T13:18:51.240Z

@jtlocsei I use clj-kondo with Cursive (using LSP) and it should just work. Do you see the little green ball in bottom right info area saying that connection to LSP server is established? Are you sure the path to your clj-kondo standalone jar is correct? Are you sure you are using stanalone version? I could have a screen sharing session with you but unfortunately I was suddenly hospitalized last weekend and now mostly laying in bed without my gallbladder and a huge incision carve in my belly and cannot sit. ๐Ÿ˜€

kari 2021-04-22T13:20:06.240200Z

Ah... I missed that he already got it working with LSP.

tobias 2021-04-22T22:32:03.240500Z

@kari.marttila I appreciate your offer of help! I got it working before I saw your comment but your comment was still useful because I hadn't noticed the green ball with the connection info. Occasionally the LSP fails to load so it's handy to be able to restart it from the green ball. Good luck with the recovery from your surgery! It sounds like a big surgery.