Skip to content
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 glide go units #1119

Merged
merged 2 commits into from
Feb 5, 2017
Merged

Fix glide go units #1119

merged 2 commits into from
Feb 5, 2017

Conversation

emilevauge
Copy link
Member

This PR fixes a regression introduced here 99f2514#diff-395f41b2a27b70ce44399c91c82158c2R36 (name instead of package).
This commit was merged because glide.lock has been manually changed, which is never a good idea. This PR also adds a check to avoid manual changes in glide.lock.
/cc @containous/traefik

@emilevauge emilevauge added this to the 1.2 milestone Feb 5, 2017
Copy link
Contributor

@vdemeester vdemeester left a comment

Choose a reason for hiding this comment

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

LGTM 😓

@emilevauge emilevauge merged commit 00de73b into master Feb 5, 2017
@guilhem
Copy link
Contributor

guilhem commented Feb 6, 2017

oops my bad for glide.yaml :(

Validation is really good things.
But it sounds pretty weird to me as I never updated glide.lock manually and don't see the problem on my local branch on glide.yaml

[edit] look like a rebase problem on #1116 :/

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

Successfully merging this pull request may close these issues.

4 participants