[
https://issues.apache.org/jira/browse/GEODE-5984?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16675681#comment-16675681
]
ASF subversion and git services commented on GEODE-5984:
--------------------------------------------------------
Commit 3d56f40ec5e9c6fa8fc0d257f28c9171732c0ce6 in geode's branch
refs/heads/feature/GEODE-5984 from [~bschuchardt]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=3d56f40 ]
GEODE-5984 Address LGTM recommendations
Addressing Kirk's initial comments
> Address LGTM recommendations
> ----------------------------
>
> Key: GEODE-5984
> URL: https://issues.apache.org/jira/browse/GEODE-5984
> Project: Geode
> Issue Type: Bug
> Reporter: Bruce Schuchardt
> Assignee: Bruce Schuchardt
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> There is a list of 87 minor "recommendations" from the LooksGoodToMe CI
> integration report for 2b4313a:
> Implicit conversion from Array to String.
> Avoid automated semicolon insertion (97% of all statements in the enclosing
> function have an explicit semicolon).
> @param for phantom parameters
> Missing spaces in concatenated string literals
> Others are in javascript files used by geode-book and Pulse.
> https://lgtm.com/projects/g/apache/incubator-geode/alerts/?mode=list&severity=recommendation
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
