babashka / neil

A CLI to add common aliases and features to deps.edn-based projects
MIT License
377 stars 27 forks source link

`neil new`: link to upstream docs for list of built in templates #165

Closed teodorlu closed 1 year ago

teodorlu commented 1 year ago

Please answer the following questions and leave the below in as part of your PR.

Should this be merged?

This PR contains proposed helptext changes incorporating suggestions from @seancorfield. I think we could get something better if we put in a bit more time.

Alternatives

166 generates helptext to include list of built in templates.

teodorlu commented 1 year ago

Reporting some intermediate conclusions.

From what I can see, there's no concept of "valid built-in template" in org.corfield.new. The create function takes a template as input. Valid template symbols have a subfolder in resources/corfield/new.

Perhaps we can just peek at the classpath to get the right args? Or we could ask for the ability to answer "what are valid templates?" from deps-new's public interface.

teodorlu commented 1 year ago

Closing in favor of #166.