Facepalm

LGTM; should someone else give this a try before we merge?

I think so

I can test this and report back.

@derekwaynecarr did you end up trying this?

@lavalamp @smarterclayton - I tried this. I think the message echo to terminal is no longer accurate with the observed results. The script refused to update anything if I had anything in my git status results - even if the content was untracked by git and never staged for commit. I see that we ignore .gitignore content in this update, but if the desire is to inspect untracked files as well, we should clarify that in the echo comment.

Untracked content should probably be flagged because the script runs git add src/ - we don't want to accidentally grab something you touched outside of it. I guess I could subset it to that path. Is not having any untracked files in your repo a burden?

@smarterclayton its not uncommon for me to keep a temp file in my workspace that has notes on the thing i am working on. It's not a major issue for me to require a clean workspace, just make the comment state that clearly "Before updating third-pary code, you must have no staged or untracked files (i.e. git status returns no results)."

@derekwaynecarr I made it only check for unclean files in third_party/src

LGTM