envoyproxy/envoy

Specify -Wdeprecated-declarations for proto targets

Open

#14 281 ouverte le 4 déc. 2020

Voir sur GitHub
 (8 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/buildhelp wantedtech debt

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

The deprecated proto fields are generating a whole lot of warnings:

bazel-out/k8-fastbuild/bin/external/envoy_api/envoy/config/core/v3/config_source.pb.h:102:3: note: declared here
  102 |   ApiConfigSource_ApiType_hidden_envoy_deprecated_UNSUPPORTED_REST_LEGACY PROTOBUF_DEPRECATED_ENUM = 0,

We should set the -Wdeprecated-declarations for proto targets to suppress this warning.

Guide contributeur

Direction de recherche
Localisez le fichier BUILD pour les cibles proto dans le depot et ajoutez ' Wdeprecated declarations' a l'attribut copts.
Stack technique
cpp
Domaine
backend
Type d'issue
Maintenance
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Bazel
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90