cljstyle icon indicating copy to clipboard operation
cljstyle copied to clipboard

Add support for configurable imports

Open nikolap opened this issue 5 years ago • 3 comments

Re https://github.com/greglook/cljstyle/issues/31

nikolap avatar Mar 04 '20 16:03 nikolap

Codecov Report

Merging #39 into master will increase coverage by 0.01%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #39      +/-   ##
==========================================
+ Coverage   92.67%   92.69%   +0.01%     
==========================================
  Files          13       13              
  Lines        1515     1519       +4     
  Branches       44       44              
==========================================
+ Hits         1404     1408       +4     
  Misses         67       67              
  Partials       44       44
Impacted Files Coverage Δ
src/cljstyle/config.clj 98.19% <100%> (+0.01%) :arrow_up:
src/cljstyle/format/ns.clj 96.67% <100%> (+0.03%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 62e1db2...e243325. Read the comment docs.

codecov-io avatar Mar 04 '20 17:03 codecov-io

Tweaked this to instead of be configurable by vector/list, simply use a flag, :vector-for-imports?

Rationale is there isn't another commonly used valid option.

Let me know if there's anything you'd like done before this can get merged

nikolap avatar Mar 10 '20 13:03 nikolap

Sorry for the delay; this needs a bit of updating to work with the latest configuration style now.

greglook avatar Mar 25 '21 04:03 greglook