Adding interactive setup menu and updating readme#300
Adding interactive setup menu and updating readme#300xtreme-steve-elliott wants to merge 1 commit intomainfrom
Conversation
dc5d5e5 to
6b70a2a
Compare
|
This is a very interesting pull request and seems to address the following concerns:
These might be real concerns. It is true that telling people to look in the opt-ins dir is kinda cheesy. That said, I am inclined to reject this PR and I want to explain why in detail. Question: is there a way to address these concerns already? Question: what are the rewards vs. risks tradeoffs?
In summary, I feel the features added in this PR are too fancy and logic-filled for the keep-it-simple ethos of this tool. Colors, numbers, formatting, prompts, parsing -- it's just too much. Alternatives and Suggestions
I welcome your thoughts. |
|
Checking for input again. |
|
I mean, sure, I could add |
If you run ./setup.sh without any arguments, it will prompt you as to whether you want to install any opt-ins, and if you respond affirmatively, then it will present you with the list of opt-ins, as read from the file names in the opt-in directory. You can provide it the numbers of the items you want to include, and it will install those.