-
Notifications
You must be signed in to change notification settings - Fork 148
Golang Dep Builder #54
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
Conversation
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.
Thanks for getting started on this. Right now my biggest concern is the regex cleaning files that is being introduced. It isn't clear why this is needed when we provide a scratch directory to work in.
tests/integration/workflows/go_dep/data/src/remote-deps/Gopkg.toml
Outdated
Show resolved
Hide resolved
@sthulb Can you rebase so we can get the build passing (you fixed the appveyor ruby in a different pr)? I will make a pass at the PR now as well. |
I'm not quite sure why ruby tests are failing for my build... I've not touched ruby in anyway in this branch. The good news is, the Go tests all passed. |
Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.