search

greglook / cljstyle

A tool for formatting Clojure code
Eclipse Public License 1.0
293 stars 39 forks source link

Add support for configurable imports #39

Open nikolap opened 4 years ago

nikolap commented 4 years ago

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

codecov-io commented 4 years ago

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.

nikolap commented 4 years ago

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

greglook commented 3 years ago

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