Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Add a hidden flag to keygen pubkey subcommand. #31192

Closed
wants to merge 1 commit into from
Closed

Add a hidden flag to keygen pubkey subcommand. #31192

wants to merge 1 commit into from

Conversation

dmakarov
Copy link
Contributor

Problem

The flag named confirm_key is checked in clap-v3-utils when pubkey is read from a remote ledger. Missing this flag triggers an error.

Summary of Changes

This change adds the flag as a hidden arg to resolve the error.

The flag named confirm_key is checked in clap-v3-utils when pubkey is
read from a remote ledger. Missing this flag triggers an error. This
change adds the flag as a hidden arg to resolve the error.
@dmakarov dmakarov requested a review from CriesofCarrots April 13, 2023 20:56
@codecov
Copy link

codecov bot commented Apr 13, 2023

Codecov Report

Merging #31192 (74d763f) into master (768405c) will decrease coverage by 0.1%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #31192     +/-   ##
=========================================
- Coverage    81.5%    81.5%   -0.1%     
=========================================
  Files         729      729             
  Lines      206304   206304             
=========================================
- Hits       168281   168249     -32     
- Misses      38023    38055     +32     

@dmakarov
Copy link
Contributor Author

fixed by #31194

@dmakarov dmakarov closed this Apr 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant