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

Include context in the vcs error message #835

Closed
wants to merge 1 commit into from
Closed

Include context in the vcs error message #835

wants to merge 1 commit into from

Conversation

guywithnose
Copy link

What does this do / why do we need it?

Adds more context to vcs errors

What should your reviewer look out for in this PR?

When vcs fails, more information is shown. For example if you run dep init -v and a vcs command times out (like #832), more information is shown.
unable to update repository: becomes unable to update repository: command killed after 2m of no activity

Do you need help or clarification on anything?

Which issue(s) does this PR fix?

This doesn't fully fix, but it's related to #832.

@guywithnose
Copy link
Author

Just realized I edited a file in the vendor folder. I'll submit to that repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants