Drop custom MrDocs from CI - #806
Merged
Merged
Conversation
MrDocs 2026.9.4 (official, non-prerelease release) fixes the segfault on GIL's CRTP-based headers that motivated pinning a personal build. The other bug that motivated it -- reference pages linking to private-member sub-pages MrDocs never emits -- is still open upstream (github.com/cppalliance/mrdocs/issues/1277). Oddly, GIL's specific repro from that issue passes cleanly against the tagged 2026.9.4 release binary itself, but fails again against the current `develop` tip (one commit newer) -- so this is unresolved/regressed, not fixed, regardless of that one data point. - docs-prerequisites / generate-doc: back to plain actions/setup-node@v4 + `bash build_antora.sh`, matching boostorg/url, redis and graph -- no cppalliance GitHub Action exists for MrDocs itself (it's resolved by the @cppalliance/antora-cpp-reference-extension npm package, not an Action), so this is the standard pattern other Boost libraries already use. - pixi.toml: drop the sdebionne/label/esrf-bcu channel and the mrdocs dependency; keeps Node + a C++ compiler for local `pixi run -e docs docs-build`, per request pixi itself stays for local dev. - build_antora.sh: drop the MRDOCS_ROOT/CONDA_PREFIX detection block. One deliberate remaining guard: local-playbook.yml keeps `failure_level: fatal` rather than `error`. Beyond #1277 itself, the extension's release selection always considers cppalliance/mrdocs' rolling `develop`/`master` tags (verified in its source: `allowMaster`/`allowDevelop` are hardcoded to `true` regardless of config, so a `version:` pin can't exclude them), so the exact MrDocs build used here isn't actually pinned and can vary from run to run. `fatal` keeps that class of cosmetic dangling-xref noise from hard-failing the build while still catching a real extraction failure. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Leftover from the interim-Doxygen phase of the MrDocs migration and never updated when MrDocs replaced it -- doxyfile doesn't even exist anymore. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #806 +/- ##
========================================
Coverage 82.85% 82.85%
========================================
Files 119 119
Lines 5634 5634
========================================
Hits 4668 4668
Misses 966 966 🚀 New features to boost your workflow:
|
sdebionne
marked this pull request as ready for review
September 7, 2026 11:57
Contributor
Author
|
The exact issue is seen here: 
|
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
1 participant
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
The documentation fails to build at the master project level:
develop dashboard link: https://www.boost.org/library/develop/gil/
master dashboard link: https://www.boost.org/library/master/gil/
Try to fix this by sticking to the template set by redis/url... projects.
Unfortunately, AFAIU, to actually test this, it needs to be merged (blindly) to develop.
Tasklist