⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@mvandeberg
Copy link
Contributor

@mvandeberg mvandeberg commented Jan 22, 2026

Summary by CodeRabbit

  • Chores
    • Improved build configuration to prevent unnecessary library dependencies from being included during documentation generation, resulting in more efficient builds.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 22, 2026

📝 Walkthrough

Walkthrough

Modified CMakeLists.txt to refine the condition for including asio and filesystem libraries in the boost corosio include set. The libraries are now excluded from the test include configuration when building MRDocs documentation, regardless of whether tests are enabled.

Changes

Cohort / File(s) Summary
Build Configuration
CMakeLists.txt
Narrowed condition for BOOST_COROSIO_INCLUDE_LIBRARIES to exclude asio and filesystem during MRDocs builds, even when tests are enabled

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

📋 A rabbit's note on configurations,
When docs are built, the tests must wait,
We narrow paths with precision,
Less clutter in the CMake state. 🐰

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: narrowing conditions for including test dependencies (asio/filesystem) to exclude them during MRDocs documentation generation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@mvandeberg mvandeberg marked this pull request as ready for review January 22, 2026 20:23
@cppalliance-bot
Copy link

An automated preview of the documentation is available at https://59.corosio.prtest3.cppalliance.org/index.html

If more commits are pushed to the pull request, the docs will rebuild at the same URL.

2026-01-22 20:28:00 UTC

@mvandeberg mvandeberg closed this Jan 22, 2026
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