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

redirect stderr to stdout for goimports #450 #451

Merged
merged 7 commits into from Oct 22, 2018

Conversation

Projects
None yet
5 participants
@u5surf
Copy link
Contributor

commented Oct 16, 2018

#450
command not found is output as stderr so, it can be redirected to stdout only using 2>&1

result :
https://travis-ci.com/u5surf/node/jobs/152053145

btw, but also installing goimports in travis, it’s failed the required dependencies test.
Is it another issue?

@u5surf u5surf requested review from tadovas and Waldz as code owners Oct 16, 2018

@Waldz Waldz requested review from vkuznecovas and zolia Oct 16, 2018

@soffokl

This comment has been minimized.

Copy link
Member

commented Oct 18, 2018

@u5surf please consider signing your commit to be able to merge it.

@u5surf u5surf dismissed stale reviews from soffokl, zolia, and Waldz via 895da8c Oct 18, 2018

@u5surf u5surf force-pushed the u5surf:issue-450-2 branch from c6b1d78 to 895da8c Oct 18, 2018

@u5surf u5surf force-pushed the u5surf:issue-450-2 branch from 895da8c to 6fd3c30 Oct 18, 2018

@Waldz

Waldz approved these changes Oct 18, 2018

@vkuznecovas

This comment has been minimized.

Copy link
Contributor

commented Oct 19, 2018

not entirely sure where first 4 commits are coming from

@Waldz Waldz merged commit 2674cce into mysteriumnetwork:master Oct 22, 2018

1 check was pending

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.