Skip to content

[WIP] tooling: Use upstream python check#18088

Closed
phlax wants to merge 1 commit intoenvoyproxy:mainfrom
phlax:tooling-upstream-python-check
Closed

[WIP] tooling: Use upstream python check#18088
phlax wants to merge 1 commit intoenvoyproxy:mainfrom
phlax:tooling-upstream-python-check

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 11, 2021

Commit Message: tooling: Use upstream python check
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 11, 2021
@repokitteh-read-only
Copy link
Copy Markdown

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).

🐱

Caused by: #18088 was opened by phlax.

see: more, trace.

@phlax phlax marked this pull request as draft September 11, 2021 21:28
@phlax phlax force-pushed the tooling-upstream-python-check branch 3 times, most recently from 8ec48a5 to 17b0bbf Compare September 11, 2021 22:19
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the tooling-upstream-python-check branch from 17b0bbf to 3936982 Compare September 14, 2021 06:54
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 14, 2021

merged with #18087

@phlax phlax closed this Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant