Fork me on GitHub
#tools-deps
<
2018-05-10
>
rickmoynihan20:05:09

@alexmiller: I’m bike shedding a little but what would you recommend calling the key for lein-tools-deps config. Currently it’s called :tools/deps which feels a little wrong, should it be :lein.tools/deps or :lein.tools.deps, or just :lein-tools-deps?

rickmoynihan21:05:09

This is the key for using in a project.clj for configuring how the lein-tools-deps plugin should work with tools.deps

Alex Miller (Clojure team)21:05:34

wait, this is in project.clj?

Alex Miller (Clojure team)21:05:14

I guess most stuff there is not namespaced

Alex Miller (Clojure team)21:05:24

saying “lein” seems redundant

Alex Miller (Clojure team)21:05:59

I guess I’d say :lein-tools-deps or :tools-deps

dominicm21:05:56

No harm in setting a new precedent.

👍 4
rickmoynihan21:05:00

yeah its in project.clj

rickmoynihan21:05:08

but thought it might be nice to set a precedent

Alex Miller (Clojure team)21:05:34

:lein-tools-deps/config

rickmoynihan21:05:08

yeah I had thought of that one too… I like that it nests config under it, but it gets wordy… though that’s not really an issue

rickmoynihan21:05:55

it also matches the actual project name on github, and root namespace name

rickmoynihan21:05:04

I think that’s definitely the most correct one