Skip to content

Conversation

dmitshur
Copy link
Member

Remove tip. It will not work, because GopherJS requires the current version of Go.

Remove tip. It will not work, because GopherJS requires the current version of Go.
@dmitshur dmitshur requested a review from dominikh July 17, 2017 22:45
@dmitshur
Copy link
Member Author

Oh, actually, while GopherJS requires current version of Go, gopherpen only uses GopherJS compiler when dev build tag is used. So it should support tip. I'll bring it back.

It should be fine, since GopherJS compiler is only used with dev build tag.
It's not used during normal build.
language: go
go:
- 1.8.x
- master
Copy link
Member

Choose a reason for hiding this comment

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

Does master mean the same thing as tip to Travis?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, it's the latest way of specifying it in Travis, that's why I changed it.

Reference: https://docs.travis-ci.com/user/languages/go/#Specifying-a-Go-version-to-use

@dmitshur
Copy link
Member Author

PTAL.

@dmitshur dmitshur merged commit 999a3c9 into master Jul 17, 2017
@dmitshur dmitshur deleted the travis-go1.8 branch July 17, 2017 23:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants