Copilot commented on code in PR #838:
URL: https://github.com/apache/iceberg-cpp/pull/838#discussion_r3637575445
##########
src/iceberg/temp_linter_probe.cc:
##########
@@ -0,0 +1,17 @@
+// TEMP verification file — will be dropped before merge.
+// Confirms cpp-linter runs clang-tidy with the pip-installed binary
+// (via --version=${pythonLocation}/bin) after dropping clang-format.
+namespace iceberg {
Review Comment:
New C++ source files in this repo consistently include the Apache Software
Foundation license header (e.g., src/iceberg/name_mapping.cc:1-18). This file
currently lacks that header, which can cause licensing/compliance checks to
fail if the file ends up being merged.
--
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]
