quickwit-oss/quickwit

Fix bad pattern used in CLI argument parsing

Open

#881 aberto em 10 de dez. de 2021

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)Rust (553 forks)github user discovery
backlogenhancementgood first issuelow-priority

Métricas do repositório

Stars
 (11.300 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

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

Guia do colaborador

Direção de pesquisa
Encontre o arquivo que contém a análise de argumentos da CLI e substitua o padrão atual pelo sugerido usando `map` e `expect`.
Pilha de tecnologia
rust
Domain
backendcli
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Menos de 1 hora
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75