Copilot commented on code in PR #634:
URL: https://github.com/apache/iceberg-cpp/pull/634#discussion_r3173302461
##########
src/iceberg/test/CMakeLists.txt:
##########
@@ -116,6 +116,7 @@ add_iceberg_test(util_test
decimal_test.cc
endian_test.cc
formatter_test.cc
+ lazy_test.cc
location_util_test.cc
Review Comment:
`lazy_test.cc` is added to the CMake `util_test` target, but the Meson test
manifest (`src/iceberg/test/meson.build`, `util_test` sources list) doesn’t
include it. This means Meson builds won’t compile/run the new test. Please add
`lazy_test.cc` to the Meson `util_test` sources as well to keep the two build
systems in sync.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
