quickwit-oss/quickwit

Fix bad pattern used in CLI argument parsing

Aperta

#881 aperta il 10 dic 2021

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (1 assegnatario)Rust (581 fork)github user discovery
backlogenhancementgood first issuelow-priority

Metriche repository

Star
 (11.537 stelle)
Metriche merge PR
 (Merge medio 5g 18h) (41 PR mergiate in 30 g)

Descrizione

As suggested by @guilload

Would be better to use this pattern:

let metastore_uri = matches
    .value_of("metastore-uri")
    .map(normalize_uri)
    .expect("`metastore-uri` should be a required arg.")?;

Guida contributor

Direzione di ricerca
Trova il file che contiene il parsing degli argomenti CLI e sostituisci il pattern corrente con quello suggerito usando `map` e `expect`.
Tech stack
rust
Dominio
backendcli
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Rust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75