clojisr icon indicating copy to clipboard operation
clojisr copied to clipboard

linter integration

Open awb99 opened this issue 5 years ago • 2 comments

I think sooner or later we will need a custom linter for clojisr. (see screenshot below).

Or what you might want to do is to generate a clojure namespace like lib python clj, which essentially allows to require R namespaces that are wrapped into clojure namespaces. Then you could at initialization time essentially build all the needed namespaces, and from there on working with R namespaces will be identical to working with clj namespaces,

line 28: base is not recognised.

linter defs

awb99 avatar Jun 01 '20 21:06 awb99

I have 600 packages in R (most of are dependencies), this requires generating huge amount of clojure code. I'm not sure if linter is possible in our case...

genmeblog avatar Jun 01 '20 21:06 genmeblog

I added a linter config to clj kondo in clojisr-gorilla: https://github.com/pink-gorilla/clojisr-gorilla/blob/master/.clj-kondo/config.edn

Unfortunately I was not successful in configuring away the R related errors. I think it is mandatory for every project to run linters. Too many errors the linter can fix; at least for me.

I don't want to add source code that has clojisr code to a folder that is exempt from linting.

Not an urgent problem. But if R code gets more and more intermingled with clj code, then it should be resolved in some way...

awb99 avatar Jun 02 '20 03:06 awb99