Skip to content

docs: Consolidate build rules#18142

Merged
wrowe merged 2 commits intoenvoyproxy:mainfrom
phlax:docs-consolidation
Sep 24, 2021
Merged

docs: Consolidate build rules#18142
wrowe merged 2 commits intoenvoyproxy:mainfrom
phlax:docs-consolidation

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Sep 16, 2021

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

Commit Message: docs: Consolidate build rules
Additional Description:

This is the first of a few PRs to cleanup/consolidate the docs build

This one:

  • updates rules_pkg to the newest version
  • makes use of the newer pkg_files and pkg_filegroup

in follow up prs i intend to:

  • consolidate the builder scripts (api, deps, extensions) into a single docs builder
  • shift the sphinx_runner to the upstream package

Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Deprecated:]
[Optional API Considerations:]

@phlax phlax marked this pull request as draft September 16, 2021 11:40
@phlax phlax force-pushed the docs-consolidation branch from b9ed526 to 97d5a82 Compare September 16, 2021 12:43
@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: #18142 was synchronize by phlax.

see: more, trace.

@repokitteh-read-only repokitteh-read-only bot added the deps Approval required for changes to Envoy's external dependencies label Sep 16, 2021
@phlax phlax marked this pull request as ready for review September 16, 2021 13:03
@phlax phlax changed the title [WIP] docs: Consolidate build rules docs: Consolidate build rules Sep 16, 2021
Signed-off-by: Ryan Northey <ryan@synca.io>
@wrowe
Copy link
Copy Markdown
Contributor

wrowe commented Sep 22, 2021

/lgtm deps

@repokitteh-read-only repokitteh-read-only bot removed the deps Approval required for changes to Envoy's external dependencies label Sep 22, 2021
wrowe
wrowe previously approved these changes Sep 22, 2021
Copy link
Copy Markdown
Contributor

@wrowe wrowe left a comment

Choose a reason for hiding this comment

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

This is fine, do you want to drop in a todo/comment about the unused/undeployed rule?

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Sep 22, 2021

do you want to drop in a todo/comment about the unused/undeployed rule?

yep, ill do it now

Signed-off-by: Ryan Northey <ryan@synca.io>
Copy link
Copy Markdown
Contributor

@wrowe wrowe left a comment

Choose a reason for hiding this comment

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

Looks good. Merging.

@wrowe wrowe merged commit 755e288 into envoyproxy:main Sep 24, 2021
soulxu pushed a commit to soulxu/envoy that referenced this pull request Oct 16, 2021
* docs: Consolidate build rules
* add-todo

Signed-off-by: Ryan Northey <ryan@synca.io>
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.

2 participants