search

gruntwork-io / boilerplate

A tool for generating files and folders ("boilerplate") from a set of templates
https://www.gruntwork.io
Mozilla Public License 2.0
160 stars 12 forks source link

Add "details" property to boilerplate variables. #11

Closed josh-padnick closed 7 years ago

josh-padnick commented 7 years ago

Consider the following

name: ElbHealthCheckType
    prompt: Enter the type for the health check (HTTP or TCP) the ELB will use to validate that a container is functioning. Use HTTP when your container returns an HTTP 200 response at var.elb_health_check_path. Use TCP when you merely want to validate that a connection can be made to the container. Valid values are HTTP, HTTPS, TCP, or SSL

That prompt is really long. I'd like to keep the prompt short and add a property like details that renders below the prompt and that's suitable for long-form text.

It'd also be nice to have something like an enum property that allows me to declare a set of valid values.

brikis98 commented 7 years ago

Agreed, this would be very handy.

As for enum, see #9.

brikis98 commented 7 years ago

In #14, I updated the way boilerplate prompts for variables to be similar to that of Terraform. I think it handles long descriptions much better now, so I'm going to close this. If you feel this needs more work, feel free to re-open.

josh-padnick commented 7 years ago

Thanks! Left some comments in #14. Either way, a definite improvement!