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

Add 'healthcheck' command to CLI #282

Merged
merged 3 commits into from Jul 9, 2018

Conversation

Projects
None yet
3 participants
@donce
Copy link
Contributor

commented Jul 5, 2018

It's inconvenient to inspect mysterium client without healthcheck.

screen shot 2018-07-05 at 11 05 12

@donce donce requested review from ignasbernotas, tadovas and Waldz Jul 5, 2018

@tadovas

This comment has been minimized.

Copy link
Member

commented Jul 5, 2018

👍

@tadovas
Copy link
Member

left a comment

💯 karma points

@donce donce force-pushed the feature/healthcheck-cli branch from a0f774d to b34a335 Jul 5, 2018

info("Build info:")
info(fmt.Sprintf(" Commit: %v", healthcheck.BuildInfo.Commit))
info(fmt.Sprintf(" Branch: %v", healthcheck.BuildInfo.Branch))
info(fmt.Sprintf(" Build number: %v", healthcheck.BuildInfo.Branch))

This comment has been minimized.

Copy link
@Waldz

Waldz Jul 5, 2018

Member

You can use BuildAsString()

This comment has been minimized.

Copy link
@tadovas

tadovas Jul 9, 2018

Member

I agree :)

donce added some commits Jul 5, 2018

@donce donce force-pushed the feature/healthcheck-cli branch from b34a335 to db5cda4 Jul 9, 2018

@tadovas

tadovas approved these changes Jul 9, 2018

Copy link
Member

left a comment

LGTM

@Waldz

Waldz approved these changes Jul 9, 2018

@tadovas tadovas merged commit 9a3bf76 into master Jul 9, 2018

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@tadovas tadovas deleted the feature/healthcheck-cli branch Jul 9, 2018

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.