eastwood

All things realted to eastwood - the Clojure linter
slipset 2019-04-28T09:16:11.013800Z

@sergeyw could I ask you to file an issue for this?

đź‘Ť 1
slipset 2019-04-28T09:16:32.014300Z

It’s not good if Eastwood craps out without really giving a good explanation.

slipset 2019-04-28T09:17:56.014800Z

Oh, and a free idea. having a linter as a nrepl middleware

slipset 2019-04-28T09:18:26.015400Z

Since nrepl has all the info about the running project, you could lint forms as they’re evaluated.

slipset 2019-04-28T09:58:06.015800Z

https://github.com/riemann/riemann/pull/944

borkdude 2019-04-28T10:24:23.016200Z

free idea for whom?

slipset 2019-04-28T10:25:16.016400Z

Anyone :)

borkdude 2019-04-28T10:28:04.016700Z

I imagine eastwood already does this?

slipset 2019-04-28T10:34:38.016900Z

Nope

slipset 2019-04-28T10:34:59.017400Z

There is no Eastwood-nrepl middleware.

slipset 2019-04-28T10:35:30.018200Z

Squiggly-Clojure, however uses Eastwood though.

borkdude 2019-04-28T10:38:01.018700Z

that’s interesting, so it doesn’t take advantage of living inside the same JVM process?

borkdude 2019-04-28T10:40:28.019300Z

it seems eastwood has copied deps. did eastwood make any changes to that code geared towards linting, or is it only to prevent classpath conflicts?

borkdude 2019-04-28T10:43:11.019600Z

it seems this answers the question: https://github.com/jonase/eastwood/tree/master/copy-deps-scripts

2019-04-28T20:45:49.020600Z

The copied deps are to enable Eastwood to lint projects that use arbitrary versions of those dependencies.

2019-04-28T20:46:17.021200Z

If a linted project happens to use exactly the same version that Eastwood does, then no need for copieddeps, but that is the exception, not the rule.

2019-04-28T20:48:43.022200Z

There are explicit warnings in Eastwood's README that one would have to be foolhardy to use Eastwood in a live running project (IMO, and I wrote that part of the README). https://github.com/jonase/eastwood#warnings-about-using-eastwood-in-a-repl

2019-04-28T20:49:20.023100Z

I mean, if you trust Eastwood that much not to introduce bugs into your compiled code, go for it, but the disclaimer is there as strongly as I know how to word it factually.