Skip to content

docs(cmd/rpccli): improve auth token documentation and CLI UX #4225

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

docs(cmd/rpccli): improve auth token documentation and CLI UX #4225

wants to merge 5 commits into from

Conversation

DeVikingMark
Copy link

Closes #3974.

Changes:

  • Improved flag descriptions for --url and --token flags
  • Enhanced error messages with clear instructions on generating tokens
  • Simplified command descriptions for better readability
  • Renamed begin-redelegate command to redelegate to match Cosmos SDK convention
  • Added comprehensive auth token documentation (docs/cli-auth-tokens.md)
  • Improved auth command output with usage examples

@github-actions github-actions bot added the external Issues created by non node team members label Apr 21, 2025
@cristaloleg cristaloleg added the kind:feat Attached to feature PRs label Apr 21, 2025
@codecov-commenter
Copy link

codecov-commenter commented Apr 21, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 22.22222% with 7 lines in your changes missing coverage. Please review.

Project coverage is 44.79%. Comparing base (2469e7a) to head (63c904e).
Report is 486 commits behind head on main.

Files with missing lines Patch % Lines
cmd/rpc.go 0.00% 5 Missing ⚠️
cmd/auth.go 0.00% 2 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #4225      +/-   ##
==========================================
- Coverage   44.83%   44.79%   -0.05%     
==========================================
  Files         265      312      +47     
  Lines       14620    22964    +8344     
==========================================
+ Hits         6555    10286    +3731     
- Misses       7313    11560    +4247     
- Partials      752     1118     +366

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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

@renaynay renaynay Awaiting requested review from renaynay renaynay is a code owner

@Wondertan Wondertan Awaiting requested review from Wondertan Wondertan is a code owner

@vgonkivs vgonkivs Awaiting requested review from vgonkivs vgonkivs is a code owner

@walldiss walldiss Awaiting requested review from walldiss walldiss is a code owner

@cristaloleg cristaloleg Awaiting requested review from cristaloleg cristaloleg is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@DeVikingMark DeVikingMark

Labels
external Issues created by non node team members kind:feat Attached to feature PRs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

docs(cmd/rpccli): ui/ux improvements
3 participants
@DeVikingMark @codecov-commenter @cristaloleg