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
Fetch remote urls #1451
Fetch remote urls #1451
Conversation
@@ -37,12 +37,17 @@ func NewEndpointFromCloneURLWithConfig(url string, c *Configuration) Endpoint { | |||
return e | |||
} | |||
|
|||
if strings.HasSuffix(url, "/") { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we already have path
imported, would it be possible to use path.Clean
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could, but it does a lot more stuff besides just chomping the trailing /
that we don't need. These urls either come from the git remote config (remote.origin.url
), or from custom lfs config (lfs.url
, remote.origin.lfsurl
).
|
||
# LFS object not downloaded, pointer in working directory | ||
refute_local_object "$contents_oid" | ||
ack "$content_oid" a.dat |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the containers that Travis gives us have ack
installed.
After you switch out |
This adds support for pushing and pulling LFS objects with a raw remote url. This was originally submitting in #1085, and updated for the current master branch. This also adds tests.