You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2022/01/11 17:29:12 UTC

[GitHub] [geode] DonalEvans commented on pull request #7221: GEODE-6588: Mass cleanup of static analyzer warnings.

DonalEvans commented on pull request #7221:
URL: https://github.com/apache/geode/pull/7221#issuecomment-1010194813


   While it's not actually gating new PRs right now due to some implementation issues, there was a vote that passed to make the LGTM pre-checkin job a gating step, so in the spirit of that, could the two remaining alerts either be fixed or suppressed as false positives (with explanatory comments) please?


-- 
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: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org