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

Always get a fresh copy of the go-licenser dep #8079

Merged
merged 1 commit into from Aug 27, 2018

Conversation

Projects
None yet
3 participants
@jalvz
Contributor

jalvz commented Aug 24, 2018

No description provided.

@jalvz jalvz requested a review from andrewkroh Aug 24, 2018

@andrewkroh

Works for me. In the future I hope to vendor the tools better.

@andrewkroh andrewkroh requested a review from ruflin Aug 24, 2018

@ruflin

ruflin approved these changes Aug 27, 2018

@ruflin ruflin merged commit 14e3110 into elastic:master Aug 27, 2018

6 checks passed

CLA Commit author is a member of Elasticsearch
Details
Hound No violations found. Woof!
beats-ci Build finished.
Details
codecov/patch Coverage not affected when comparing bf72246...5d93056
Details
codecov/project 64.42% (-0.06%) compared to bf72246
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ruflin

This comment has been minimized.

Show comment
Hide comment
@ruflin

ruflin Aug 27, 2018

Collaborator

++ on vendoring

Collaborator

ruflin commented Aug 27, 2018

++ on vendoring

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