clj-kondo

https://github.com/clj-kondo/clj-kondo
martinklepsch 2020-08-19T12:51:58.033500Z

@borkdude I tried the updated hooks now and it’s a huuuuuge improvement πŸ‘ Only last issue I’m having is that the docstring seems to be expected in the wrong place, according to the docs it should appear after the component name: https://p22.tr4.n0.cdn.getcloudapp.com/items/yAubDXb2/Image%202020-08-19%20at%202.50.45%20PM.png?v=ec0644f5c4dcb74d72953b01901f963c

fiddlerwoaroof 2020-08-19T17:55:06.034Z

fiddlerwoaroof 2020-08-19T17:55:24.034700Z

Is this lint message wrong?

fiddlerwoaroof 2020-08-19T17:55:35.035Z

It looks like clj-kondo doesn't recognize custom transducers

borkdude 2020-08-19T17:56:26.035200Z

@martinklepsch What's a docstring?

πŸ˜† 2
borkdude 2020-08-20T10:08:03.000900Z

@martinklepsch Fixed it on master

lread 2020-08-20T12:48:46.003100Z

Yay @martinklepsch! And thanks for letting me know, I have crossed it off my todo list!

martinklepsch 2020-08-21T12:15:28.012400Z

Thank youuuu! πŸ™Œ I tried digging into the hooks DSL a little bit a few weeks ago and had a hard time getting started 😬 πŸ˜…

borkdude 2020-08-21T12:17:08.012600Z

Hopefully more examples makes it easier. It surely helps if you have used rewrite-clj before

borkdude 2020-08-19T17:56:47.035300Z

@martinklepsch Kidding. That's probably not too hard to fix. If you want, you can make an issue, or if you feel inclined, a PR

borkdude 2020-08-19T17:57:39.035600Z

Yeah. Made an issue for it here: https://github.com/borkdude/clj-kondo/issues/952

fiddlerwoaroof 2020-08-19T17:57:51.036Z

Cool, thanks

borkdude 2020-08-19T17:58:36.036200Z

You can override the type config, if it's bothering you enough right now

fiddlerwoaroof 2020-08-19T17:58:50.036400Z

It's fine for now, this is in my "sandbox" project

lread 2020-08-19T19:04:39.036600Z

I thought you were glitching there for a second! :simple_smile:

borkdude 2020-08-19T19:05:27.036800Z

It was clickbait to seduce you to do a PR

lread 2020-08-19T19:10:34.037Z

Nice marketing!