envoyproxy/envoy

format: add checks for type aliases

Open

#9.840 geöffnet am 27. Jan. 2020

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/buildbeginnerenhancementhelp wanted

Repository-Metriken

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

Beschreibung

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.

Contributor Guide

Research-Richtung
Das Issue schlägt vor, automatisierte Prüfungen für die in STYLE.md definierten Konventionen für Typaliase hinzuzufügen. Die Hauptdateien, die untersucht werden müssen, sind check format.py im Format CI des Repositorys, und möglicherweise die Erweiterung von clang tidy mit benutzerdefinierten Prüfungen. Überprüfen Sie STYLE.md auf die spezifischen Konventionen und sehen Sie sich die vorhandenen Formatprüfungen in check format.py an, um das Muster zu verstehen. Die Diskussion im Issue könnte zusätzlichen Kontext zum bevorzugten Ansatz liefern.
Tech Stack
cpp
Domain
backend
Issue Type
Funktion
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.
1-2 Tage
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.
Meist klar
Voraussetzungen
GitC++
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70