envoyproxy / envoyproxy/envoy

Specify -Wdeprecated-declarations for proto targets

Open
#14,281 8 comments 0 reactions 0 assignees View on GitHub
area/build help wanted tech debt
Dominant language
C++
Stars
28.9k
Forks
5.6k
Avg merge
1d 20h
Merged PRs (30d)
437

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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.