Skip to content

tooling: Use upstream abstracts#17823

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

tooling: Use upstream abstracts#17823
phlax wants to merge 1 commit intoenvoyproxy:mainfrom
phlax:tooling-upstream-abstracts

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Aug 24, 2021

Signed-off-by: Ryan Northey ryan@synca.io

Commit Message: tooling: Use upstream abstracts
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 Aug 24, 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: #17823 was opened by phlax.

see: more, trace.

@moderation
Copy link
Copy Markdown
Contributor

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Aug 24, 2021
@phlax phlax force-pushed the tooling-upstream-abstracts branch 2 times, most recently from 8c6fa84 to c38d542 Compare August 26, 2021 09:27
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Aug 26, 2021
@phlax phlax force-pushed the tooling-upstream-abstracts branch from c38d542 to e272262 Compare August 26, 2021 09:35
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the tooling-upstream-abstracts branch from e272262 to 4f77088 Compare August 26, 2021 09:37
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Aug 26, 2021

closed in favour of #17843

@phlax phlax closed this Aug 26, 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.

2 participants