Fork me on GitHub
#clj-kondo
<
2019-06-16
>
sogaiu00:06:05

@borkdude why did you decide to use a .bat with lein as you mentioned here: https://github.com/taylorwood/clj.native-image/pull/9#issuecomment-497981891? like you, i shaved all of the yaks you seemed to, to get a deps.edn / native-image build to work w/ windows 10. was it that it seems to be less work for someone to replicate the lein / .bat approach?

sogaiu00:06:30

mmm, actually, i thought i was using clj via compile, but no, compile resolves to compile.bat, and compile is a bash script and wouldn't have run anyway...sorry, for the noise.

sogaiu02:06:29

@borkdude on a related note, regarding: https://github.com/taylorwood/clj.native-image/pull/9#issue-278357391 if the windows clj / clojure came with a standalone .exe / .com / .bat to launch the clojure-tooling implemented via powershell, do you think the lein / .bat approach would still be necessary?

borkdude06:06:48

@sogaiu I ran into too many issues with a GraalVM clj wrapper that a .bat was easier

sogaiu07:06:56

@borkdude hmm...it seems like it's almost there though...you also filed TDEPS-128 iiuc...that one seems to imply that a standalone launcher for the clojure-tooling would be helpful. does that seem right?

borkdude07:06:29

yeah, almost, but that wasn’t the case when I worked on it 🙂

borkdude07:06:30

see #clj-on-windows

borkdude07:06:00

I think a GraalVM executable for clj/clojure would make sense

sogaiu08:06:54

was just having similar thoughts within the last few hours :) but i don't envy the folks having to do the windows building! btw, i summarized related info here: https://gist.github.com/sogaiu/e079cd770051685c46ab24b6658effcf

borkdude08:06:53

nice, thanks

lread19:06:30

Cool @marc-omorain! Friendly feedback: you might want to state which editor/ide this is an extension for in your github project desription and readme (Visual Studio Code, right?)

Marc O'Morain19:06:04

Thanks 🙏 The readme needs a LOT of work 👍

Marc O'Morain14:06:06

Version 0.1 published 🎉

borkdude15:06:31

w00t!!! 😄 awesome!

borkdude15:06:21

how doth one install this extension?

borkdude15:06:14

I’m about to release a new version of clj-kondo. I’ll announce the extension together with it

Marc O'Morain15:06:49

There is a GUI for installing them. It’s named “Clojure-lint” (I hope!)

Marc O'Morain15:06:19

I renamed the repo. Pro tip: don’t do too much typing on a MacBook Pro.

Marc O'Morain15:06:29

The keyboard is terrible.

borkdude15:06:22

So I heard (>= 2016 models)

borkdude15:06:04

I’m preparing a new release and will announce your work together with it.

borkdude16:06:05

and thank you!

borkdude16:06:40

clj-kondo v2019.06.16-alpha! 🎉 major new features: - unresolved symbol detection - VSCode integration (thank you @marc-omorain !) https://github.com/borkdude/clj-kondo/releases/tag/v2019.06.16-alpha

aw_yeah 12
Marc O'Morain18:06:35

@borkdude about the info level: https://github.com/marcomorain/clojure-lint/blob/de399b6fba3931649113b66754ac1dbd09d6ed42/src/extension.ts#L64 I was looking for an exhaustive list of levels in the Kondo source but I couldn’t spot it.

Marc O'Morain18:06:55

Fixing the info issue is a letter of filling out this switch

borkdude18:06:06

@marc-omorain That’s the only missing level

Marc O'Morain18:06:38

The literal string info right?

borkdude18:06:16

yes. you might want to map it to Hint or Info in VSCode, but Hint doesn’t exist in other editors always, so that’s why I chose info

Marc O'Morain18:06:18

My stretch goal is to add “suppress this” as an auto fix, so that users can click on a warning, and i would create a minimal patch to their .clj-kondo/config.edn` will be made to suppress that warning.

borkdude18:06:38

That would be very nice.

Marc O'Morain18:06:55

vscode makes it simple to add actions to each diagnostic.

borkdude18:06:59

I would have to be extra careful not to break anything in the config format, which is my stretch goal 🙂

Marc O'Morain18:06:03

The action itself is the hard part 😭

Marc O'Morain18:06:01

Don’t worry about breaking the config format - extensions for vscode are simple to update, so I can react very quickly to any changes.

borkdude18:06:33

do you expect people to upgrade their extension e.g. once a week, or do they update automatically?

Marc O'Morain18:06:16

It just happens automatically

borkdude18:06:39

I was thinking of this feature: https://github.com/borkdude/clj-kondo/issues/253 so people know if something unexpected is in their config and they can upgrade, so I have more freedom to deprecate something (which I will try to avoid, but still)

borkdude18:06:53

(very much inspired by the CircleCI CLI)

borkdude18:06:54

Also, feel free to make a PR for the editor integration docs for VSCode (if I don’t beat you to it)

Marc O'Morain18:06:20

That’s really meta

Marc O'Morain18:06:32

Linting the config - I love it.

borkdude18:06:05

yeah, I might also add support for linting project.clj or deps.edn 🙂

Stefan20:06:44

@marc-omorain @borkdude clj-kondo in VSCode, sweet guys!!! 🎉