quickwit-oss/quickwit

Fix bad pattern used in CLI argument parsing

Offen

#881 geöffnet am 10.12.2021

Auf GitHub ansehenContributor Guide
 (1 Kommentar) (0 Reaktionen) (1 zugewiesene Person)Rust (581 Forks)github user discovery
backlogenhancementgood first issuelow-priority

Repository-Metriken

Stars
 (11.537 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 5T 18h) (41 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Finden Sie die Datei mit der CLI Argumentanalyse und ersetzen Sie das aktuelle Muster durch das vorgeschlagene mit `map` und `expect`.
Tech Stack
rust
Domain
backendcli
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Rust
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75