Skip to content
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

Use pipes.quote when displaying command #848

Merged
merged 1 commit into from
Jun 19, 2018
Merged

Use pipes.quote when displaying command #848

merged 1 commit into from
Jun 19, 2018

Conversation

asottile
Copy link
Contributor

WIP for #845

Some example output:

fix-lint runtests: commands[0] | pre-commit run --all-files --show-diff-on-failure
...
fix-lint runtests: commands[1] | python -c 'print("hint: run /tmp/tox/.tox/fix-lint/bin/pre-commit install to add checks as pre-commit hook")'
...

@codecov
Copy link

codecov bot commented Jun 19, 2018

Codecov Report

Merging #848 into master will increase coverage by <1%.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master   #848    +/-   ##
======================================
+ Coverage      92%    92%   +<1%     
======================================
  Files          13     13            
  Lines        2414   2415     +1     
  Branches      428    428            
======================================
+ Hits         2214   2215     +1     
  Misses        124    124            
  Partials       76     76

@gaborbernat gaborbernat merged commit 47454a7 into tox-dev:master Jun 19, 2018
@asottile asottile deleted the pipes_quote branch June 19, 2018 06:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants