[
https://issues.apache.org/jira/browse/DRILL-7856?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17303717#comment-17303717
]
ASF GitHub Bot commented on DRILL-7856:
---------------------------------------
vvysotskyi commented on pull request #2187:
URL: https://github.com/apache/drill/pull/2187#issuecomment-801436062
I prefer a fewer number of PRs if possible. Sometimes it is required to look
through the commits history or commit annotations in the class to find a commit
with some changes. Also, it is easier for the reviewer to review similar
changes in a single PR, and it will be possible to point out additional places
with similar issues.
----------------------------------------------------------------
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)
