Skip to content

ci: Add CI workflow to check docs build in PRs - #985

Merged
koverholt merged 5 commits into
mainfrom
ci-docs-build-check
Nov 20, 2025
Merged

ci: Add CI workflow to check docs build in PRs#985
koverholt merged 5 commits into
mainfrom
ci-docs-build-check

Conversation

@koverholt

@koverholt koverholt commented Nov 19, 2025

Copy link
Copy Markdown
Collaborator

This PR adds a GitHub action / workflow to build the docs in a PR as a pass/fail check.

It runs with mkdocs build --strict to fail on any warnings, and will ensure that a given branch / PR will not fail the docs build if merged, which could lead to silent failures until manually caught and fixed.

@koverholt

koverholt commented Nov 19, 2025

Copy link
Copy Markdown
Collaborator Author

Tested with an example breakage to mkdocs.yml in 3cc926b that linked to a non-existent .md file, and the test failed as expected.

@joefernandez

Copy link
Copy Markdown
Collaborator

@koverholt - Let's monitor this closely for a few days to make sure this does not significantly impact forward movement of content updates.

@koverholt

Copy link
Copy Markdown
Collaborator Author

@joefernandez, will do. Note that this will not block any current or future PR merges, only show up a failing test. We would need to add a ruleset to enforce this, which we don't need to do at this time. So you can consider this a "soft" CI test for PRs.

@koverholt
koverholt merged commit 457afb5 into main Nov 20, 2025
4 checks passed
@koverholt
koverholt deleted the ci-docs-build-check branch November 20, 2025 00:54
koverholt added a commit to sabrenner/google-adk-docs that referenced this pull request Apr 28, 2026
* Add CI workflow to check docs build in PRs

* Intentionally break mkdocs.yml to check CI test

* Revert intentional config breakage / test

* Test CI run
koverholt added a commit to sabrenner/google-adk-docs that referenced this pull request Apr 28, 2026
* Add CI workflow to check docs build in PRs

* Intentionally break mkdocs.yml to check CI test

* Revert intentional config breakage / test

* Test CI run
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