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

run: forward SIGHUP to subprocess #48

Merged
merged 1 commit into from
Jul 21, 2018
Merged

run: forward SIGHUP to subprocess #48

merged 1 commit into from
Jul 21, 2018

Conversation

blueyed
Copy link
Member

@blueyed blueyed commented Jul 21, 2018

This is necessary to quit Vim in case input/output is closed.

Ref: AndrewRadev/vimrunner#51 (comment)

@codecov
Copy link

codecov bot commented Jul 21, 2018

Codecov Report

Merging #48 into master will decrease coverage by 0.01%.
The diff coverage is 97.05%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
- Coverage   97.36%   97.35%   -0.02%     
==========================================
  Files          15       15              
  Lines        1520     1548      +28     
  Branches      165      165              
==========================================
+ Hits         1480     1507      +27     
- Misses         27       28       +1     
  Partials       13       13
Flag Coverage Δ
#py27 97.09% <97.05%> (-0.02%) ⬇️
#py34 95.73% <97.05%> (+0.01%) ⬆️
#py35 96.57% <97.05%> (-0.01%) ⬇️
#py36 96.57% <97.05%> (-0.01%) ⬇️
Impacted Files Coverage Δ
tests/test_cli.py 100% <100%> (ø) ⬆️
covimerage/cli.py 94.11% <90.9%> (-0.33%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2e2785f...62d2f45. Read the comment docs.

This is necessary to quit Vim in case input/output is closed.

Ref: AndrewRadev/vimrunner#51 (comment)
@blueyed blueyed merged commit df962b4 into Vimjas:master Jul 21, 2018
@blueyed blueyed deleted the sighup branch July 21, 2018 12:00
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.

1 participant