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 19:18:47 UTC

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

pivotal-jbarrett commented on pull request #7221:
URL: https://github.com/apache/geode/pull/7221#issuecomment-1010283330


   > 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?
   
   Both of those have been suppressed, following their Java suppression guide, and tickets for the false-positive issues have been opened. Unfortunately it looks like their Java suppression technique is also broken. 


-- 
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