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 streaming response parser so server progress messages aren't delayed (for #106) #116

Merged
merged 1 commit into from
Apr 1, 2018

Conversation

billiegoose
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Apr 1, 2018

Codecov Report

Merging #116 into master will decrease coverage by 0.05%.
The diff coverage is 89.47%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #116      +/-   ##
==========================================
- Coverage   90.13%   90.07%   -0.06%     
==========================================
  Files          64       64              
  Lines        1581     1592      +11     
  Branches      294      297       +3     
==========================================
+ Hits         1425     1434       +9     
- Misses        135      137       +2     
  Partials       21       21
Impacted Files Coverage Δ
src/managers/GitRemoteHTTP.js 88.67% <100%> (-0.11%) ⬇️
src/models/GitPktLine.js 91.66% <84.61%> (-8.34%) ⬇️

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 5ce7f1c...14e4d04. Read the comment docs.

@billiegoose billiegoose merged commit 903136d into master Apr 1, 2018
@billiegoose billiegoose deleted the streaming-parser branch April 1, 2018 03:54
@billiegoose
Copy link
Member Author

🎉 This PR is included in version 0.9.8 🎉

The release is available on:

Your semantic-release bot 📦🚀

modesty pushed a commit to modesty/isomorphic-git that referenced this pull request Apr 9, 2024
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