Skip to content

wasm: update proxy-wasm-cpp-host to use refactored code around runtimes.#16162

Merged
lizan merged 2 commits intoenvoyproxy:mainfrom
mathetake:refactored-cpp-host
Apr 27, 2021
Merged

wasm: update proxy-wasm-cpp-host to use refactored code around runtimes.#16162
lizan merged 2 commits intoenvoyproxy:mainfrom
mathetake:refactored-cpp-host

Conversation

@mathetake
Copy link
Copy Markdown
Member

Signed-off-by: Takeshi Yoneda takeshi@tetrate.io

Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Apr 26, 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: #16162 was opened by mathetake.

see: more, trace.

PiotrSikora
PiotrSikora previously approved these changes Apr 26, 2021
Copy link
Copy Markdown
Contributor

@PiotrSikora PiotrSikora left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
@mathetake
Copy link
Copy Markdown
Member Author

kindly ping @envoyproxy/dependency-shepherds

@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 Apr 27, 2021
@lizan lizan merged commit 6c7777c into envoyproxy:main Apr 27, 2021
gokulnair pushed a commit to gokulnair/envoy that referenced this pull request May 6, 2021
…es. (envoyproxy#16162)

Signed-off-by: Takeshi Yoneda <takeshi@tetrate.io>
Signed-off-by: Gokul Nair <gnair@twitter.com>
@mathetake mathetake deleted the refactored-cpp-host branch June 29, 2021 04:18
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.

4 participants