Conversation
|
An automated preview of the documentation is available at https://71.openmethod.prtest3.cppalliance.org/libs/openmethod/doc/html/index.html If more commits are pushed to the pull request, the docs will rebuild at the same URL. 2026-04-26 22:50:40 UTC |
There was a problem hiding this comment.
Pull request overview
This PR consolidates/updates the CMake-based integration test setup for Boost.OpenMethod to follow the “unified cmake test” pattern (tracking the referenced Boost.URL layout), replacing the previous cmake_subdir_test / cmake_install_test scaffolding.
Changes:
- Add a unified
test/cmake_test/CMake project that supports multiple CI modes (install test, module install test, Boost superproject subdir test, and standalone add_subdirectory test). - Add a CMake-driven example program (
test/cmake_test/main.cpp) exercisingBOOST_OPENMETHODdispatch. - Remove the older
test/cmake_subdir_test/*andtest/cmake_install_test/*CMake test files.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| test/cmake_test/main.cpp | Adds the unified CMake test executable source demonstrating openmethod dispatch. |
| test/cmake_test/CMakeLists.txt | Adds the unified CMake test driver with multiple CI configuration modes. |
| test/cmake_subdir_test/main.cpp | Removed in favor of the unified cmake_test program. |
| test/cmake_subdir_test/CMakeLists.txt | Removed in favor of unified multi-mode CMake logic. |
| test/cmake_install_test/CMakeLists.txt | Removed; install/module install covered by unified cmake_test logic. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #71 +/- ##
========================================
Coverage 94.34% 94.34%
========================================
Files 43 43
Lines 2916 2916
========================================
Hits 2751 2751
Misses 165 165 Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot apply changes based on the comments in this thread |
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Track https://github.com/boostorg/url/blob/develop/test/cmake_test/CMakeLists.txt