-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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 show-address command #2170
Add show-address command #2170
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2170 +/- ##
==========================================
- Coverage 63.62% 63.5% -0.13%
==========================================
Files 136 136
Lines 8270 8286 +16
==========================================
Hits 5262 5262
- Misses 2652 2668 +16
Partials 356 356 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small note on the doc update, but otherwise, utACK 👍
docs/validators/security.md
Outdated
@@ -39,3 +39,14 @@ Sentry Nodes should edit their config.toml: | |||
# Comma separated list of peer IDs to keep private (will not be gossiped to other peers) | |||
private_peer_ids = "ipaddress of validator nodes" | |||
``` | |||
|
|||
## Environment Variables |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did this come from?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ohhh, Looks like this address #1973. Wanna tag that somewhere (at least in the PR body)
server/tm_cmds.go
Outdated
return cmd | ||
} | ||
|
||
func printCryptoWire(data interface{}) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd call it printlnJSON(v interface{}) error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice
* Testnet upgraded to v9 * Update docs/hub-tutorials/join-testnet.md Co-authored-by: yaruwangway <69694322+yaruwangway@users.noreply.github.com> * Update docs/hub-tutorials/join-testnet.md Co-authored-by: yaruwangway <69694322+yaruwangway@users.noreply.github.com> --------- Co-authored-by: yaruwangway <69694322+yaruwangway@users.noreply.github.com>
Closes #1274.
Targeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added entries in
PENDING.md
with issue #rereviewed
Files changed
in the github PR explorerFor Admin Use: