[ 
https://issues.apache.org/jira/browse/GRIFFIN-312?focusedWorklogId=362950&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-362950
 ]

ASF GitHub Bot logged work on GRIFFIN-312:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 24/Dec/19 09:22
            Start Date: 24/Dec/19 09:22
    Worklog Time Spent: 10m 
      Work Description: wankunde commented on issue #560: [GRIFFIN-312] Code 
Style Standardization
URL: https://github.com/apache/griffin/pull/560#issuecomment-568704176
 
 
   @chitralverma 
   
   LGTM
   
   `scalafmt` plugin is more strict with scala code than 
`scalastyle-maven-plugin`.
   Could you also update the develop document 
`https://github.com/apache/griffin/blob/master/griffin-doc/dev/code-style.md` 
for new developers.
   
   Thanks.
 
----------------------------------------------------------------
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]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 362950)
    Time Spent: 40m  (was: 0.5h)

> Code Style Standardization
> --------------------------
>
>                 Key: GRIFFIN-312
>                 URL: https://issues.apache.org/jira/browse/GRIFFIN-312
>             Project: Griffin
>          Issue Type: Improvement
>            Reporter: Chitral Verma
>            Priority: Major
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> This issue targets to,
>  * fix the various warnings during build and in source code,
>  * perform code formatting as per a standard style,
>  * fix scalastyle integration
> Using the [code format from spark code style can be automatically imposed on 
> measure module. Link: 
> [https://github.com/apache/spark/blob/master/dev/.scalafmt.conf]
> Since ScalaStyle targets scala source code only, it should be a part of the 
> measure module only. Current misconfiguration is also suppressing the 
> formatting errors 
> Scalafmt is used for code formatting. 
> Note these styles are fully configurable and can be changed at later points 
> of time after discussion in dev community.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to