github / training-utils

A series of Bash and ZShell scripts useful when teaching about git and GitHub

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Update install script(s)

patrickmckenna opened this issue Β· comments

@github/training-teachers We don't currently list the dependencies needed by some of these scripts, or have any scripts to install those deps. (I didn't even realize this until this morning, when I couldn't run add-collaborators because I recently reinstalled OS X and didn't have octokit installed.)

Question the first: should we add that? I vote yes.

Question two: should we just modify the Rakefile, or should we follow the conventions of Scripts to Rule Them All? (We'd omit the unneeded bits from the latter, of course, e.g. script/console.) I vote for the second path, for consistency's sake, something that's especially important for new users and new hires.

Thoughts?

πŸ‘ for adding that
πŸ‘ for dependencies listed in scripts. especially making sure the folder is /script, and ./script/bootstrap would bail if octokit wasn't setup, etc

@patrickmckenna - thank you so much for your πŸ‘€ for these details.

Question the first: should we add that?

πŸ‘

should we follow the conventions of Scripts to Rule Them All?

I πŸ‘ option B