envoyproxy/envoy

fix clang-tidy reported errors on master

Open

#4863 aperta il 26 ott 2018

Vedi su GitHub
 (15 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
beginnerhelp wantedtech debt

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Description: clang-tidy runs against diffs in CI, but it will caught all errors on the lines the PR author touched. Ideally we should make a full clang-tidy run error free.

Guida contributor

Direzione di ricerca
Configura l'ambiente di build di envoy, esegui clang tidy con la configurazione del progetto e correggi ogni errore segnalato. Controlla i log CI per esempi di errori.
Tech stack
cpp
Dominio
backend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
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
C++clang tidyGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60