zhjwpku commented on PR #32: URL: https://github.com/apache/iceberg-cpp/pull/32#issuecomment-2612040409
> I have never configured the cpp-linter action previously but don't we have to update the [tidy-checks option](https://cpp-linter.github.io/cpp-linter-action/inputs-outputs/#tidy-checks) to an empty string in order to use that config file? From the docs: > > > It is also possible to rely solely on a .clang-tidy config file by specifying this option as a blank string (''). > > My question is if this is currently being exercised and enforced on our linter CI step I checked that while working on this PR, both '' and file works. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
