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

Improve validator signer authorization examples #7612

Merged

Conversation

kphed
Copy link
Contributor

@kphed kphed commented Apr 5, 2021

Description

Add missing blsKey and blsPop flags to validator signer authorization examples in 3 places:

  1. releasegold:authorize CLI command help text
  2. Baklava "Running a validator" docs
  3. Mainnet "Running a validator" docs

This change helps users avoid the following error when authorizing a new validator signer:

Other changes

N/A

Tested

Ran the CLI binary locally and successfully viewed the updated command help text.

Related issues

N/A

Backwards compatibility

Change is backwards compatible.

Documentation

Copy link
Contributor

@eelanagaraj eelanagaraj left a comment

Choose a reason for hiding this comment

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

LGTM! will update docs now

Copy link
Contributor

@eelanagaraj eelanagaraj left a comment

Choose a reason for hiding this comment

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

Whoops didn't approve before

@eelanagaraj eelanagaraj added the automerge Have PR merge automatically when checks pass label Apr 23, 2021
@mergify mergify bot merged commit a388944 into celo-org:master Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Have PR merge automatically when checks pass
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants