Skip to content

test: disable docs build in CI, for now - #32741

Merged
kdmccormick merged 1 commit into
openedx:masterfrom
kdmccormick:kdmccormick/stop-building-docs
Jul 13, 2023
Merged

test: disable docs build in CI, for now#32741
kdmccormick merged 1 commit into
openedx:masterfrom
kdmccormick:kdmccormick/stop-building-docs

Conversation

@kdmccormick

Copy link
Copy Markdown
Member

@feanil is overhauling the docs structure of edx-platform currently. Recently, a change he made upped the docs build time from <15mins to >40mins. Rather than revert the changes he is making, we're going to take the docs build check out of CI for now. He plans to re-enable the build check, but as a ReadTheDocs build, not a GitHub Action. He's actively working on the docs, so he's not worried about any regressions that may pop up while the check is disabled.

It's worth noting that the current docs check does not actually publish its output anywhere.

FYI @jmbowman

@feanil is overhauling the docs structure of edx-platform currently.
Recently, a change he made upped the docs build time from <15mins
to >40mins. Rather than revert the changes he is making, we're going
to take the docs build check out of CI for now. He plans to re-enable
the build check, but as a ReadTheDocs build, not a GitHub Action.
He's actively working on the docs, so he's not worried about any
regressions that may pop up while the check is disabled.

It's worth noting that the current docs check does not actually
publish its output anywhere.
@kdmccormick
kdmccormick merged commit 24130fe into openedx:master Jul 13, 2023
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@kdmccormick
kdmccormick deleted the kdmccormick/stop-building-docs branch July 13, 2023 20:32
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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.

3 participants