Skip to content

Conversation

@phlax
Copy link
Member

@phlax phlax commented Oct 14, 2024

No description provided.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Oct 14, 2024
@repokitteh-read-only
Copy link

CC @envoyproxy/dependency-shepherds: Your approval is needed for changes made to (bazel/.*repos.*\.bzl)|(bazel/dependency_imports\.bzl)|(api/bazel/.*\.bzl)|(.*/requirements\.txt)|(.*\.patch).
envoyproxy/dependency-shepherds assignee is @mattklein123

🐱

Caused by: #36568 was opened by phlax.

see: more, trace.

@phlax phlax changed the title deps/protoprocess: Resolve compiler warnings [WIP] deps/protoprocess: Resolve compiler warnings Oct 14, 2024
@phlax phlax marked this pull request as draft October 14, 2024 12:48
@phlax
Copy link
Member Author

phlax commented Oct 14, 2024

upstream PR is here grpc-ecosystem/proto_processing_lib#5

cc @dchakarwarti @asedeno

running this here to ensure it resolves the issue/s and doesnt break anything, but if upstream is not landed before release then i think we should land this

@phlax
Copy link
Member Author

phlax commented Oct 14, 2024

seems to fix the problem and doesnt break anything (related)

@phlax phlax added this to the 1.32.0 milestone Oct 15, 2024
@phlax phlax force-pushed the deps-patch-proto_processing branch from 9c80c94 to 37e9087 Compare October 15, 2024 10:23
@phlax phlax changed the title [WIP] deps/protoprocess: Resolve compiler warnings deps/protoprocess: Resolve compiler warnings Oct 15, 2024
@phlax phlax marked this pull request as ready for review October 15, 2024 10:57
@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Oct 15, 2024
@alyssawilk alyssawilk merged commit 87f3921 into envoyproxy:main Oct 15, 2024
21 checks passed
grnmeira pushed a commit to grnmeira/envoy that referenced this pull request Oct 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants