chore: include docs build scripts in pipeline + fix scripts#6913
Merged
jakovljevic-mladen merged 1 commit intoReactiveX:masterfrom Mar 9, 2023
Conversation
Member
|
Let me know if you need anything for this. |
9177c2e to
db8b22b
Compare
Member
Author
|
Changes from the commit 1685d86 were suggested by this comment. |
Member
Author
This was referenced May 2, 2022
c87a0db to
f271f13
Compare
eb081d8 to
c79e8b5
Compare
benlesh
approved these changes
Mar 7, 2023
Member
|
@jakovljevic-mladen feel free to merge these into 7.x and master as you please. |
jakovljevic-mladen
added a commit
that referenced
this pull request
Mar 9, 2023
(cherry picked from commit afb1c66)
Member
Author
|
Cherry-picked to One note: changes from this PR only will now include running |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description:
To avoid issues that we had multiple times so far where the docs build was broken after some PR merges, I'm adding running docs build and generation scripts to the pipelines. This only runs
setupandbuildscripts from thedocs_app, it does not deploy app remotely.Related issue (if exists):
None, but one of the latest issue we had was fixed in #6881. We want to avoid having such PRs by including these runs. Also, running current
buildscript will fail pipelines until #6912 is merged.