Skip to content

docs: Move docs/build.sh to bazel#15229

Merged
htuch merged 1 commit intoenvoyproxy:mainfrom
phlax:docs-bazel-build
Jun 22, 2021
Merged

docs: Move docs/build.sh to bazel#15229
htuch merged 1 commit intoenvoyproxy:mainfrom
phlax:docs-bazel-build

Conversation

@phlax
Copy link
Copy Markdown
Member

@phlax phlax commented Feb 28, 2021 •
edited
Loading

Commit Message: docs: Move docs/build.sh to bazel
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #13229
[Optional Deprecated:]
[Optional API Considerations:]

@repokitteh-read-only
Copy link
Copy Markdown

repokitteh-read-only bot commented Feb 28, 2021

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: #15229 was opened 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 Feb 28, 2021
@phlax phlax marked this pull request as draft February 28, 2021 12:58
@phlax phlax force-pushed the docs-bazel-build branch 4 times, most recently from bf42599 to e6c9fdd Compare March 1, 2021 10:44
@alyssawilk
Copy link
Copy Markdown
Contributor

alyssawilk commented Mar 1, 2021

ping back when this is ready for review!

@phlax phlax force-pushed the docs-bazel-build branch 6 times, most recently from 4defa66 to 341399c Compare March 5, 2021 06:15
@mattklein123 mattklein123 self-assigned this Mar 14, 2021
@phlax phlax force-pushed the docs-bazel-build branch from 341399c to d7df173 Compare March 15, 2021 04:22
@phlax phlax force-pushed the docs-bazel-build branch from 205126d to dd31c21 Compare March 15, 2021 04:33
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Mar 15, 2021

/retest

@repokitteh-read-only
Copy link
Copy Markdown

repokitteh-read-only bot commented Mar 15, 2021

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #15229 (comment) was created by @phlax.

see: more, trace.

phlax
phlax commented Mar 16, 2021
View reviewed changes
phlax
phlax commented Mar 16, 2021
View reviewed changes
phlax
phlax commented Mar 16, 2021
View reviewed changes
phlax
phlax commented Mar 16, 2021
View reviewed changes
phlax
phlax commented Mar 16, 2021
View reviewed changes
@phlax phlax force-pushed the docs-bazel-build branch from 8919ffc to 73b6b62 Compare April 7, 2021 14:42
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Jun 12, 2021

coverage is here https://storage.googleapis.com/envoy-pr/15229/tooling/index.html

@phlax phlax force-pushed the docs-bazel-build branch 5 times, most recently from d762cd9 to 192c578 Compare June 13, 2021 12:08
This was referenced Jun 13, 2021
Merged
Merged
@phlax phlax force-pushed the docs-bazel-build branch from 192c578 to 544869b Compare June 13, 2021 13:02
@phlax phlax mentioned this pull request Jun 13, 2021
Merged
@phlax phlax force-pushed the docs-bazel-build branch 5 times, most recently from 1f96888 to 20d9c96 Compare June 14, 2021 13:38
phlax
phlax commented Jun 15, 2021
View reviewed changes
@phlax phlax force-pushed the docs-bazel-build branch from 20d9c96 to ff44487 Compare June 15, 2021 23:04
phlax
phlax commented Jun 15, 2021
View reviewed changes
phlax
phlax commented Jun 15, 2021
View reviewed changes
phlax
phlax commented Jun 15, 2021
View reviewed changes
phlax
phlax commented Jun 15, 2021
View reviewed changes
phlax
phlax commented Jun 15, 2021
View reviewed changes
phlax
phlax commented Jun 15, 2021
View reviewed changes
@phlax phlax mentioned this pull request Jun 16, 2021
Closed
@phlax
Copy link
Copy Markdown
Member Author

phlax commented Jun 21, 2021

ping back when this is ready for review!

@alyssawilk ping!

@htuch this is the final part in the move to building docs with bazel - should be ready for review!

@alyssawilk
Copy link
Copy Markdown
Contributor

alyssawilk commented Jun 21, 2021

Still looks like a draft? per multiple requests I'm going to switch our tooling to ignore them, so maybe de-draft?
I'll let @htuch take a look either way since he's reviewer and Matt's out.

@phlax
Copy link
Copy Markdown
Member Author

phlax commented Jun 21, 2021

Still looks like a draft?

oops - done

@phlax
docs: Shift build to bazel and python
37643d2 
Signed-off-by: Ryan Northey <ryan@synca.io>
htuch
htuch approved these changes Jun 22, 2021
View reviewed changes
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, nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@dmitri-d dmitri-d dmitri-d left review comments

@htuch htuch htuch approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@adisuissa adisuissa

@mattklein123 mattklein123

@htuch htuch

Labels

api deps Approval required for changes to Envoy's external dependencies

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Make it easier to build docs

6 participants

@phlax @alyssawilk @dmitri-d @htuch @adisuissa @mattklein123