quickwit-oss/quickwit

Fix bad pattern used in CLI argument parsing

Ouverte

#881 ouverte le 10 déc. 2021

Voir sur GitHubGuide contributeur
 (1 commentaire) (0 réaction) (1 personne assignée)Rust (581 forks)github user discovery
backlogenhancementgood first issuelow-priority

Métriques du dépôt

Stars
 (11 537 étoiles)
Métriques de merge PR
 (Merge moyen 5j 18h) (41 PRs mergées en 30 j)

Description

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.")?;

Guide contributeur

Direction de recherche
Trouvez le fichier contenant l'analyse des arguments CLI et remplacez le modèle actuel par celui suggéré en utilisant `map` et `expect`.
Stack technique
rust
Domaine
backendcli
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Rust
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75