-
-
Notifications
You must be signed in to change notification settings - Fork 10.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dev-cmd/sh.rb: non-interactive mode #8482
Conversation
f581831
to
61ef58a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice idea, good work so far!
61ef58a
to
a810f61
Compare
@MikeMcQuaid good to merge? |
Last question: wondering if anyone would want to do/add |
0749366
to
41589cc
Compare
Made one final tweak. Thanks again @maxim-belkin! |
8ecdaa0
to
483fbb7
Compare
|
|
I can fix this by adding |
fecb1fd
to
cd075dd
Compare
Thanks @maxim-belkin! |
brew style
with your changes locally?brew tests
with your changes locally?brew sh
, e.g.:ENV["VERBOSE"] = "1"
withif args.verbose?