I am seeing this:
|
(def default-paths ["src" "test" "project.clj"]) |
But the project.clj file is ignored when running lein cljfmt check (empty configuration), unless I specifically specify it (lein cljfmt check project.clj).
I've narrowed it down to (seq paths) never being empty when the paths come from the Leiningen plugin:
|
paths (or (seq paths) (filter file-exists? default-paths))] |
Do you want a PR changing lein-cljfmt to always include the project.clj file?
I was thinking that it could look something like (changing lein-cljfmt/src/leiningen/cljfmt.clj):
- (filter #(and (.exists %) (.isDirectory %)))))))
+ (filter #(and (.exists %) (.isDirectory %)))
+ (concat '("project.clj"))))))
It seems to be a good fit since the project file is lein specific anyway.
I am seeing this:
cljfmt/cljfmt/src/cljfmt/main.clj
Line 169 in 1003c67
But the
project.cljfile is ignored when runninglein cljfmt check(empty configuration), unless I specifically specify it (lein cljfmt check project.clj).I've narrowed it down to
(seq paths)never being empty when the paths come from the Leiningen plugin:cljfmt/cljfmt/src/cljfmt/main.clj
Line 216 in 1003c67
Do you want a PR changing
lein-cljfmtto always include theproject.cljfile?I was thinking that it could look something like (changing
lein-cljfmt/src/leiningen/cljfmt.clj):It seems to be a good fit since the project file is lein specific anyway.