Skip to content

ci: add GitHub Action to build and validate documentation - #490

Merged
apaleyes merged 2 commits into
EmuKit:mainfrom
apaleyes-bot:add-docs-build-workflow
Jul 26, 2026
Merged

ci: add GitHub Action to build and validate documentation#490
apaleyes merged 2 commits into
EmuKit:mainfrom
apaleyes-bot:add-docs-build-workflow

Conversation

@apaleyes-bot

Copy link
Copy Markdown
Contributor

Add a minimal 'Build docs' workflow that runs on every push and pull request to ensure Sphinx documentation builds without errors.

What this catches

  • Missing or broken Sphinx directives
  • Broken cross-references
  • Missing doc dependencies
  • Documentation build failures early in the CI process

Workflow details

  • Installs Python 3.10
  • Installs full dev dependencies (which includes Sphinx and related packages)
  • Runs make html in the doc directory
  • Fails if any build errors occur

Comment thread .github/workflows/docs.yml Outdated
push:
branches: [ main ]
pull_request:
branches: [ main ]

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Let's run for all PRs

Comment thread .github/workflows/docs.yml Outdated
Comment thread .github/workflows/docs.yml Outdated
@apaleyes-bot
apaleyes-bot force-pushed the add-docs-build-workflow branch 2 times, most recently from cfb95b4 to 9c6b564 Compare July 26, 2026 16:09
@codecov

codecov Bot commented Jul 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.24%. Comparing base (7cd414c) to head (2037bd9).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #490      +/-   ##
==========================================
+ Coverage   89.10%   89.24%   +0.13%     
==========================================
  Files         137      137              
  Lines        4847     4861      +14     
  Branches      479      477       -2     
==========================================
+ Hits         4319     4338      +19     
+ Misses        402      396       -6     
- Partials      126      127       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Add a minimal 'Build docs' workflow that runs on every push and pull request
to ensure Sphinx documentation builds without errors. This catches:
- Missing or broken Sphinx directives
- Broken cross-references
- Missing doc dependencies
- Documentation build failures early in the CI process

Co-authored-by: Andrei Paleyes <2852301+apaleyes@users.noreply.github.com>
@apaleyes-bot
apaleyes-bot force-pushed the add-docs-build-workflow branch from 9c6b564 to a3b14cb Compare July 26, 2026 17:04
@apaleyes-bot
apaleyes-bot force-pushed the add-docs-build-workflow branch 6 times, most recently from 07f85e1 to 6067bee Compare July 26, 2026 20:28
Comment thread doc/conf.py Outdated
Comment thread emukit/core/loop/outer_loop.py Outdated
Fix docstring indentation and formatting issues:
- gpy_model_wrappers.py: Fix indentation in get_covariance_between_points
- convert_lists_to_array.py: Fix docstring indentation for convert_y_list_to_array and convert_xy_lists_to_arrays
- non_linear_multi_fidelity_model.py: Fix math directive spacing in make_non_linear_kernels

Fix reStructuredText formatting:
- outer_loop.py: Fix inline code markup (backticks) in docstring
- installation.rst: Fix title underlines and code block indentation
- emukit.quadrature.measures.rst: Fix title underline length

Fix cross-references and missing documents:
- tutorials.rst: Fix references to nonexistent notebook names (add 'tutorial-' prefix)
- local_search_acquisition_optimizer.py: Remove duplicate footnote labels
- wsabi.py: Remove duplicate footnote reference labels
- sixhumpcamel.py: Fix math directive spacing and formula typo (= -> +)

Fix notebook formatting:
- Emukit-tutorial-Bayesian-quadrature-introduction.ipynb: Fix inconsistent heading levels (#### -> ###)

These changes resolve all 66 documentation warnings/errors reported by Sphinx with -W flag.

Co-authored-by: Andrei Paleyes <2852301+apaleyes@users.noreply.github.com>
@apaleyes-bot
apaleyes-bot force-pushed the add-docs-build-workflow branch from 6067bee to 87b32d0 Compare July 26, 2026 21:30
@apaleyes
apaleyes merged commit ffd2517 into EmuKit:main Jul 26, 2026
10 checks passed
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