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
fix: Registry cache for Azure DevOps #1690
Conversation
Signed-off-by: Jakub <13458283+diznq@users.noreply.github.com>
I think it's ok to put a dependency on |
URL: r.url.String(), | ||
}) | ||
if err != nil { | ||
return errors.Wrap(err, "cloning remote registry") |
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.
Could such error handling have been more perfect?
Co-authored-by: Joe Kutner <jpkutner@gmail.com> Signed-off-by: Jakub <13458283+diznq@users.noreply.github.com>
Co-authored-by: Joe Kutner <jpkutner@gmail.com> Signed-off-by: Jakub <13458283+diznq@users.noreply.github.com>
Strange, did you make mistakes with the rebase, why are there multiple commits? |
Signed-off-by: Jakub <13458283+diznq@users.noreply.github.com>
I just signed off the comments raised here but realized strings import wasn't removed and Go is strict about unused stuff |
This indicates that recent changes have not achieved the desired coverage, and more work needs to be done to increase coverage. |
Summary
Go Git doesn't work with Azure DevOps due to unsupported protocol: go-git/go-git#64
Output
If
dev.azure.com/
URL is used for Git,git clone
is used instead of Go Git library, the downside isgit
must be installed locally