-
Notifications
You must be signed in to change notification settings - Fork 96
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
improved shell handling #181
Conversation
@jish so how do you like it? I have few more other PRs waiting also ;) |
options = args.last.is_a?(::Hash) ? args.pop : {} | ||
args = build_command(*args) | ||
execute_raw(args, options) | ||
end |
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.
Seems cleaner in all of the check files. Any reason not to use Open3
anymore?
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.
with Open3
we get separate stdout
from stderr
, with the 2>&1
it's merged properly, even if it is possible to get it ordered properly with open3 - it would take more code to that, less code has to be better ;)
@jish did you had time to review it further? |
Conflicts: lib/plugins/pre_commit/checks/coffeelint.rb lib/plugins/pre_commit/checks/scss_lint.rb lib/pre-commit/checks/grep.rb
merged master after #146 |
No description provided.