paleolimbot commented on code in PR #615: URL: https://github.com/apache/arrow-nanoarrow/pull/615#discussion_r1764351623
########## docs/source/roadmap.md: ########## @@ -42,6 +41,10 @@ have not yet been scheduled for implementation. however, including them in the default test suite complicates release verification for some users and prevents testing in environments where Arrow C++ does not currently build (e.g., WASM, compilers without C++17 support). +- **Test verbosity**: Tests for the C library were written before testing utilities + in the `nanoarrow_testing` library were available (and before there was a + `nanoarrow_testing` library in which to put new ones). As a result, some of them + are very verbose and can be difficult to read. W Review Comment: 😬 💠-- 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]
