drill-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [drill] eevanwong commented on pull request #2187: DRILL-7856 Add lgtm badge to Drill and fix alerts
Date Wed, 17 Mar 2021 17:16:02 GMT

eevanwong commented on pull request #2187:
URL: https://github.com/apache/drill/pull/2187#issuecomment-801261325

   @vvysotskyi yeah for sure. A big thing is that we want to make sure that what we're changing/surpressing
isnt too major. So with each alert (or file w/ alerts) I have a small justification of why
I surpressed/changed it. It'd be really cluttered if we put them all on 3 big prs. Would you
prefer it if we had smaller prs, or just do as many as possible?

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.

For queries about this service, please contact Infrastructure at:

View raw message