Skip to content
This repository was archived by the owner on Sep 11, 2020. It is now read-only.

packp/capability: trim spaces on capabilities decode. #170

Merged
merged 1 commit into from
Dec 9, 2016
Merged

packp/capability: trim spaces on capabilities decode. #170

merged 1 commit into from
Dec 9, 2016

Conversation

smola
Copy link
Collaborator

@smola smola commented Dec 9, 2016

git 1.8 used to write a leading space in the capabilities
announcement for git-receive-pack. We now trim spaces before
decoding a capabilities list to be more tolerant about
this kind of difference.

git 1.8 used to write a leading space in the capabilities
announcement for git-receive-pack. We now trim spaces before
decoding a capabilities list to be more tolerant about
this kind of difference.
@codecov-io
Copy link

codecov-io commented Dec 9, 2016

Current coverage is 76.97% (diff: 100%)

Merging #170 into master will decrease coverage by 0.74%

@@             master       #170   diff @@
==========================================
  Files            88         88          
  Lines          5493       5494     +1   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
- Hits           4269       4229    -40   
- Misses          765        814    +49   
+ Partials        459        451     -8   

Powered by Codecov. Last update 654e54d...f53e722

@mcuadros mcuadros merged commit 963c190 into src-d:master Dec 9, 2016
@smola smola deleted the capabilities-leading-space branch December 9, 2016 11:59
mcuadros pushed a commit that referenced this pull request Jan 31, 2017
git 1.8 used to write a leading space in the capabilities
announcement for git-receive-pack. We now trim spaces before
decoding a capabilities list to be more tolerant about
this kind of difference.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants