envoyproxy/envoy

format: add checks for type aliases

Open

#9840 aperta il 27 gen 2020

Vedi su GitHub
 (2 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/buildbeginnerenhancementhelp wanted

Metriche repository

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

Descrizione

Description: Envoy has several conventions around type aliases defined in STYLE.md. Right now these conventions are checked by reviewers. It would be really useful to have the conventions checked in the format CI run.

This could be done with regexes in the check_format.py script, or optimally by extending clang-tidy.

Guida contributor

Direzione di ricerca
La segnalazione propone di aggiungere controlli automatizzati per le convenzioni degli alias di tipo definite in STYLE.md. I file principali da esaminare sono check format.py nel CI di formattazione del repository, e potenzialmente estendere clang tidy con controlli personalizzati. Rivedere STYLE.md per le convenzioni specifiche e osservare i controlli di formattazione esistenti in check format.py per comprenderne lo schema. La discussione nella segnalazione potrebbe fornire ulteriore contesto sull'approccio preferito.
Tech stack
cpp
Dominio
backend
Tipo issue
Funzionalità
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.
Abbastanza chiara
Prerequisiti
GitC++
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70