envoyproxy/envoy

Prevent addition of redundant protobuf imports

Open

#15.589 geöffnet am 20. März 2021

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
area/buildhelp wanted

Repository-Metriken

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

Beschreibung

description

we have a lot of redundant import warnings when generating protos

i have raised a pr to fix this upstream in cncf/xds (https://github.com/cncf/xds/pull/3)

i have previously removed them from the envoy codebase (#15329 ) and have just opened another PR to remove more recently added ones (#15588 )

it would be great if we prevented these from being added

the obvious way would be to turn warnings into errors (see https://github.com/protocolbuffers/protobuf/issues/3980)

the problem with this is that there are a lot of warnings about missing dir paths which i think is harder to fix (https://github.com/bazelbuild/bazel/issues/7157)

Contributor Guide

Research-Richtung
Untersuchen Sie, wie zwischen redundanten Importwarnungen und fehlenden Verzeichniswarnungen in Protobuf unterschieden werden kann. Lesen Sie die referenzierten Issues und PRs für den Kontext. Erwägen Sie, das Build System (Bazel) so zu ändern, dass redundante Importwarnungen als Fehler behandelt werden, während fehlende Verzeichniswarnungen erlaubt bleiben.
Tech Stack
Keine
Domain
backendinfrastructurebuild system
Issue Type
Wartung
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
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
GitBazelprotobuf
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40