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 CI tests #498

Merged
merged 10 commits into from
Apr 18, 2016
Merged

Fix CI tests #498

merged 10 commits into from
Apr 18, 2016

Conversation

fd0
Copy link
Member

@fd0 fd0 commented Apr 16, 2016

No description provided.

return nil
}

func (env *AppveyorEnvironment) Teardown() error {

Choose a reason for hiding this comment

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

exported method AppveyorEnvironment.Teardown should have comment or be unexported

@fd0 fd0 force-pushed the fix-travis-drawin branch 2 times, most recently from fb937c1 to 39613de Compare April 17, 2016 17:14

env.goxOS = []string{"linux", "darwin", "freebsd", "openbsd", "windows"}
func (env *TravisEnvironment) Teardown() error {

Choose a reason for hiding this comment

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

exported method TravisEnvironment.Teardown should have comment or be unexported

@fd0 fd0 force-pushed the fix-travis-drawin branch 2 times, most recently from 439402b to 3efee19 Compare April 17, 2016 19:23
return run("go", "run", "build.go", "-v", "-T")
}

func (env *AppveyorEnvironment) Teardown() error {

Choose a reason for hiding this comment

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

exported method AppveyorEnvironment.Teardown should have comment or be unexported

@fd0 fd0 merged commit 2233890 into master Apr 18, 2016
fd0 added a commit that referenced this pull request Apr 18, 2016
@fd0 fd0 deleted the fix-travis-drawin branch April 18, 2016 19:03
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.

None yet

2 participants