Open thekungfuspider opened 1 year ago
@thekungfuspider This is great.
Though I'm not to keen on flags for users who are not familiar with terminal and CLi's.
How about we use basic option menus if no flags are passed. I have made some pseudo examples
pseudo_lpx_link_options_spec.rb
The specs don't cover the number options.
What say you?
@davidteren - thanks for the feedback and drawing up that example! I agree with you that for beginners, CLI options can be confusing / not user friendly when troubleshooting. So a prompt based approach might be better, to help beginners modularly use the various features provided. I like the approach you've taken re the defaulting to prompt if no CLI options are provided. An interactive prompt would vastly improve the ease of use. After taking a look I have a couple questions:
@davidteren - bump on this. If the answer is yes to both questions, I'll go ahead and make the change for your review.
@thekungfuspider I've been giving this some thought and have started investigating ways to run this from a UI.
I think hold off on your efforts for now. I should have time to work on it this week.
@thekungfuspider and other users. I apologize for not making progress with the new version. I'm busy wrapping up at my current job and moving to a new role in the last week of April. I should have more time going forward.
2 issues in focus here:
If none of these options are provided, the script will execute all 3 steps. If any are provided, only the steps inputted will be executed.
In addition, the wrapper script will accept the following options: -h | --help -m | --mandatory-packages (default) -a | --all-packages
At the end of the day, we'll be able to simplify the README to:
ALL PACKAGES
cd ~/Downloads; mkdir -p lpx_links/app ; cd lpx_links/app ; curl -#L https://goo.gl/nUrpPi | tar -xzv --strip-components 1 ; sudo ./wrapper_script.sh -a