envoyproxy/envoy

presubmit-check the use of DECLARE_FACTORY for each DEFINE_FACTORY

Open

#23.173 aperta il 19 set 2022

Vedi su GitHub
 (3 commenti) (1 reazione) (0 assegnatari)C++ (5373 fork)batch import
enhancementhelp wanted

Metriche repository

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

Descrizione

E-M apparently depends on the former and nothing checks on it upstream :-(

Guida contributor

Direzione di ricerca
Analizza le macro DEFINE FACTORY e DECLARE FACTORY esistenti nel codice. Determina il pattern per garantire che ogni DEFINE FACTORY abbia un corrispondente DECLARE FACTORY. Proponi un controllo pre commit (ad esempio, usando uno script o un job CI) che scansiona le violazioni.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
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++understanding of Envoy macro system
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
35