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

Include consensus mechanism in status #1483

Merged
merged 1 commit into from Jan 20, 2019

Conversation

Projects
None yet
2 participants
@bkase
Copy link
Contributor

bkase commented Jan 20, 2019

coda client status tells you the current consensus mechanism

@bkase bkase requested a review from jkrauska Jan 20, 2019

@jkrauska
Copy link
Contributor

jkrauska left a comment

lgtm

what about 'proof of steak'?

@bkase

This comment has been minimized.

Copy link
Contributor Author

bkase commented Jan 20, 2019

Haha I like that, but I think it’s better to mirror the exact name that’s in the config file for copy paste purposes

@bkase bkase merged commit c397ddf into master Jan 20, 2019

10 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
ci/circleci: build Your tests passed on CircleCI!
Details
ci/circleci: build-macos Your tests passed on CircleCI!
Details
ci/circleci: build_public Your tests passed on CircleCI!
Details
ci/circleci: build_withsnark Your tests passed on CircleCI!
Details
ci/circleci: test-all_sig_integration_tests Your tests passed on CircleCI!
Details
ci/circleci: test-all_stake_integration_tests Your tests passed on CircleCI!
Details
ci/circleci: test-unit-test Your tests passed on CircleCI!
Details
ci/circleci: test-withsnark Your tests passed on CircleCI!
Details
ci/circleci: tracetool Your tests passed on CircleCI!
Details

@jkrauska jkrauska deleted the consensus-status branch Jan 22, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment