quickwit-oss/quickwit

Fix bad pattern used in CLI argument parsing

Open

#881 geöffnet am 10. Dez. 2021

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

Repository-Metriken

Stars
 (11.300 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

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