envoyproxy/envoy

presubmit-check the use of DECLARE_FACTORY for each DEFINE_FACTORY

Open

#23.173 geöffnet am 19. Sept. 2022

Auf GitHub ansehen
 (3 Kommentare) (1 Reaktion) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

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

Contributor Guide

Research-Richtung
Analysieren Sie die vorhandenen DEFINE FACTORY und DECLARE FACTORY Makros im Code. Ermitteln Sie das Muster, um sicherzustellen, dass jedes DEFINE FACTORY ein entsprechendes DECLARE FACTORY hat. Schlagen Sie eine Vorab Commit Prüfung vor (z. B. mit einem Skript oder CI Job), die nach Verstößen sucht.
Tech Stack
cpp
Domain
backendinfrastructure
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.
Halber Tag
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
C++understanding of Envoy macro system
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
35