[
https://issues.apache.org/jira/browse/DRILL-7856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17303590#comment-17303590
]
ASF GitHub Bot commented on DRILL-7856:
---------------------------------------
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:
[email protected]
> Add lgtm badge to Drill and fix alerts
> --------------------------------------
>
> Key: DRILL-7856
> URL: https://issues.apache.org/jira/browse/DRILL-7856
> Project: Apache Drill
> Issue Type: Improvement
> Components: Documentation
> Affects Versions: 1.18.0
> Reporter: Vitalii Diravka
> Priority: Trivial
> Labels: badge, github
>
> Consider adding new badges to Drill github, for instance _lgtm_ badges (code
> quality and alerts number):
> [https://lgtm.com/projects/g/apache/drill/context:java]
> As an example please check:
> [https://github.com/kaitoy/pcap4j]
> As a separate ticket can be considered decreasing the number of alerts of
> Drill project:
> https://lgtm.com/projects/g/apache/drill/alerts/?mode=list
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
