Skip to content

[WIP] docs: Use genrule to generate extension db#16481

Closed
phlax wants to merge 4 commits into
envoyproxy:mainfrom
phlax:docs-gen-extension-db
Closed

[WIP] docs: Use genrule to generate extension db#16481
phlax wants to merge 4 commits into
envoyproxy:mainfrom
phlax:docs-gen-extension-db

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented May 13, 2021

Commit Message: docs: Use genrule to generate extension db
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 May 13, 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: #16481 was opened by phlax.

see: more, trace.

@phlax phlax force-pushed the docs-gen-extension-db branch 2 times, most recently from d88e5bb to c64efac Compare May 13, 2021 15:58
@phlax phlax changed the title [WIP] docs: Use genrule to generate extension db docs: Use genrule to generate extension db May 13, 2021
@phlax phlax marked this pull request as ready for review May 13, 2021 15:59
@phlax phlax requested a review from htuch May 13, 2021 15:59
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This seems kind of hacky; is this going to be robust?

Copy link
Copy Markdown
Member

@htuch htuch left a comment

Choose a reason for hiding this comment

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

LGTM, is this relpath business the solution to finding SRCDIR?

phlax added 2 commits May 16, 2021 12:28
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax force-pushed the docs-gen-extension-db branch from c64efac to 0bdc12e Compare May 16, 2021 14:36
@phlax phlax changed the title docs: Use genrule to generate extension db [WIP] docs: Use genrule to generate extension db May 16, 2021
@phlax phlax marked this pull request as draft May 16, 2021 14:37
@phlax
Copy link
Copy Markdown
Member Author

phlax commented May 16, 2021

awaiting #16496

phlax added 2 commits May 16, 2021 16:11
Signed-off-by: Ryan Northey <ryan@synca.io>
Signed-off-by: Ryan Northey <ryan@synca.io>
@phlax phlax closed this May 27, 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