Skip to content

Conversation

erizocosmico
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Aug 29, 2017

Codecov Report

Merging #152 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #152   +/-   ##
=======================================
  Coverage   69.73%   69.73%           
=======================================
  Files          11       11           
  Lines         935      935           
=======================================
  Hits          652      652           
  Misses        214      214           
  Partials       69       69

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 c84451a...af23d16. Read the comment docs.

glide.yaml Outdated
@@ -12,7 +12,7 @@ import:
version: 763351934333e34370292b308c74bcb71035e22a
repo: https://github.com/src-d/go-billy-siva.git
- package: gopkg.in/src-d/go-git.v4
version: 773841734e11f0f607f6b047235f8527d475538f
version: 7aa9d15d395282144f31a09c0fac230da3f65360
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this commit is tagged in go-git as "v4.0.0-rc14", maybe it's better use the tag name instead the commit here so you can see which go-git version is being used at a glance.

What do you think?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure if the tag is reachable from gopkg.in

@erizocosmico
Copy link
Contributor Author

erizocosmico commented Aug 29, 2017

Don't merge til we update it to have src-d/go-git#569

@erizocosmico erizocosmico merged commit 815fd7d into src-d:master Aug 29, 2017
@erizocosmico erizocosmico deleted the update-go-git branch August 29, 2017 11:55
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.

4 participants