Fork me on GitHub
#clj-kondo
<
2019-11-07
>
whilo01:11:07

btw. i have also had another fun static analysis project some time ago: https://github.com/whilo/beichte/. i am not sure how important verified functional purity is to users. it would have to be triggered by metadata, i guess. e.g. putting a purity flag on all defns.

coltnz09:11:01

is it just me or does the clj-kondo annotations not work with cursive anymore. I see the warnings in the log "warning: redundant let" but nothing on the code

coltnz09:11:15

on intellij 2019.2.4 and cursive eap8

borkdude09:11:59

I'm not in control of any changes to IntelliJ but I'm not aware of any changes to clj-kondo which should break existing editor integration

borkdude09:11:40

@coltnz you can try older versions of clj-kondo or older versions of intellij/cursive to be sure

coltnz09:11:37

ah ive found it, filewatchers got somehow disabled in inspections listing

coltnz09:11:51

thx for kondo its great

❤️ 4
delaguardo12:11:48

wrote simple github action to annotate diffs with findings from clj-kondo: https://github.com/DeLaGuardo/clojure-lint-action

borkdude12:11:52

@delaguardo my mind explodes!

delaguardo12:11:29

this is docker-based action atm. So you can not use build tools to calculate classpath for linting. But I will switch it to the hosted solution in the next release.

delaguardo12:11:13

also I’m really appreciate any kind of feedback)

borkdude12:11:32

I'll mention this in the release notes of the next release

borkdude13:11:25

@delaguardo if it's easier in any way, you can also use the clj-kondo uberjar from github and run it using java

borkdude13:11:42

(I don't see how that is easier than downloading the binary, but the option is there)

borkdude13:11:09

Oh, running the uberjar also makes it work for Windows builds I guess

borkdude13:11:58

or if it's easier, you can just write a tools.deps/clojure script while using clj-kondo as a lib and invoke it that way

delaguardo13:11:56

binary or uberjar (I’m more on that approach, because it should work everywhere) a bit easier and also makes github tools caching possible

delaguardo16:11:20

I did not even knew that issues might be disabled) And this issue is fixed by the way

borkdude16:11:11

I added a comment to the issue.

borkdude16:11:42

@delaguardo STDERR should be ignored for JSON parsing. It only is used to print exceptions and such.

delaguardo16:11:12

Thanks, I’m sure it is ignored. And only used for logging when everything goes bad.

borkdude16:11:29

oh of course, I misread 🙂

borkdude16:11:50

removed my comment

delaguardo16:11:43

anyway - thanks for noticing)

zane18:11:21

Oh my god.

zane18:11:33

This is so incredibly cool, @delaguardo.

zane18:11:49

Making a note to try to get this running on our projects.

sogaiu21:11:45

lol, just finished listening to the clojurescript podcast for clj-kondo -- some of the content is already somewhat out-of-date because of the speed of dev 🙂

sogaiu21:11:16

specifically, snap support discontinued, potemkin import-vars support done(?), atom editor support, and some type-checking at least have happened since...

borkdude21:11:18

haha, I can't even remember what features I've mentioned back then. it was recorded in August