[ 
https://issues.apache.org/jira/browse/CASSANDRA-18618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17736126#comment-17736126
 ] 

Brandon Williams edited comment on CASSANDRA-18618 at 6/23/23 2:14 PM:
-----------------------------------------------------------------------

-Removing the j8 gate on checkstyle will reintroduce the problem in 
CASSANDRA-18423.-

Jacek pointed out to me that we won't be running checkstyle with tests and this 
patch, so it will actually solve CASSANDRA-18423.


was (Author: brandon.williams):
Removing the j8 gate on checkstyle will reintroduce the problem in 
CASSANDRA-18423.

> Update tasks configuration to run checks locally when requested
> ---------------------------------------------------------------
>
>                 Key: CASSANDRA-18618
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18618
>             Project: Cassandra
>          Issue Type: Task
>          Components: Build
>            Reporter: Jacek Lewandowski
>            Assignee: Jacek Lewandowski
>            Priority: Normal
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> Currently CheckStyle and RAT are run with almost every single Ant target, 
> which is annoying as when developing locally. The targets should be clear - 
> "test" - runs the tests, "jar" - builds the project and creates jars, and 
> then we should have a task "check" which runs all the static analysis, that 
> is CheckStyle, RAT and Eclipse-Warnings (or whatever we decide to replace 
> Eclipse-Warnings with).
> Such goal should be include in "artifacts" and we should run it instead of 
> "eclipse-warnings" on CircleCI. This way building, static analysis and 
> testings are clearly separated.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to