Skip to content

fix: Prevent pushing broken docs to main#719

Merged
jnumainville merged 9 commits intodeephaven:mainfrom
jnumainville:ui_fail_autodoc
Aug 15, 2024
Merged

fix: Prevent pushing broken docs to main#719
jnumainville merged 9 commits intodeephaven:mainfrom
jnumainville:ui_fail_autodoc

Conversation

@jnumainville
Copy link
Copy Markdown
Collaborator

  1. Plumbs through error codes so make_docs.py fails correctly
  2. make_docs.yml now runs on pull request to main (without sync) so broken docs have to be cleaned up before merge
  3. Fixed text_area.py so docs are able to build

Tested on my fork

Comment thread .github/workflows/build-main-docs.yml
Comment thread .github/workflows/make-docs.yml
Comment thread .github/workflows/modified-plugin.yml Outdated
Comment thread sphinx_ext/make_docs_utilities.py Outdated
Comment thread sphinx_ext/make_docs_utilities.py Outdated
Comment thread sphinx_ext/make_docs_utilities.py Outdated
Comment thread sphinx_ext/make_docs_utilities.py Outdated
Comment thread sphinx_ext/make_docs_utilities.py
Comment thread sphinx_ext/make_docs_utilities.py
Comment thread sphinx_ext/make_docs_utilities.py Outdated
@jnumainville jnumainville requested a review from mofojed August 15, 2024 17:10
@jnumainville
Copy link
Copy Markdown
Collaborator Author

@mofojed any further thoughts?

@jnumainville jnumainville merged commit 86fb7aa into deephaven:main Aug 15, 2024
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