Fork me on GitHub
#clj-kondo
<
2019-12-04
>
borkdude17:12:12

New intermediate binary version for linting missing requires (like (foo/bar) and (reduce foo/bar ...) when there is no require for foo): Mac: https://7556-176829714-gh.circle-artifacts.com/0/release/clj-kondo-2019.11.24-SNAPSHOT-macos-amd64.zip Linux: https://7555-176829714-gh.circle-artifacts.com/0/release/clj-kondo-2019.11.24-SNAPSHOT-linux-amd64.zip Branch: missing-require Please report any false positives you will find.

borkdude17:12:32

I found one problem in project.clj where there are names like foo/bar used in the defproject macro. If this is a common problem with other macros, then I will probably have to support a similar config as unresolved-symbol.

dominicm17:12:03

ah yes, macros. They ruin everything.

celwell17:12:40

Hi, I have clj-kondo working from the command line, thanks; but I'm having trouble getting it to integrate with flycheck in emacs. I installed flycheck-clj-kondo from MELPA and have it required in my init.el, but the syntax checkers seem to be disabled.

celwell18:12:13

Oh, hmmm... maybe clj-kondo isn't properly on my path. It works in Terminal (macos) but is not available in M-x shell

sogaiu18:12:35

@celwell i had trouble with flycheck at one point -- here are some brief tips that i wrote down once i got things working better: https://gist.github.com/sogaiu/5f93faa7cab1d08a1510d281cadb0b81

👍 4
celwell18:12:10

Yay, I got it working. For future reference, this is what I added to my Emacs init.el:

(setenv "PATH" (concat (getenv "PATH") ":/usr/local/bin"))
(setq exec-path (append exec-path '("/usr/local/bin")))

sogaiu18:12:12

congratulations 🙂 i think the clj-kondo flycheck docs might benefit from some troubleshooting tips

sogaiu18:12:02

for reference, i took the circuitous route of moving some PATH stuff to ~/.profile

sogaiu18:12:36

and making other arrangements for .profile to be read in other shell-related init files

borkdude18:12:09

PR welcome!

borkdude18:12:32

oh already one in progress I see 🙂

👍 4
borkdude19:12:31

merged

🙂 4