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.

Sign up for GitHub

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

Jump to bottom

Use 'master' instead of 'tip' for Travis config #22

Merged
merged 1 commit into from
May 31, 2017
Merged

Use 'master' instead of 'tip' for Travis config #22

merged 1 commit into from
May 31, 2017

Conversation

iafan
Copy link
Contributor

@iafan iafan commented May 31, 2017

iafan referenced this pull request May 31, 2017
@iafan
An attempt to fix Travis CI builds and add modern Go targets
99fca80 
Reason: `go vet` is no longer go-get-able and is available
as a standard tool since Go 1.6. I think it should be fine to drop
Go 1.4 from test targets for this reason.
dmitshur
dmitshur approved these changes May 31, 2017
View reviewed changes
Copy link
Contributor

@dmitshur dmitshur left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for following up!

@sqs sqs merged commit bd320f5 into sourcegraph:master May 31, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dmitshur dmitshur dmitshur approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@iafan @dmitshur @sqs