[
https://issues.apache.org/jira/browse/CASSANDRA-18618?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17743725#comment-17743725
]
Jacek Lewandowski commented on CASSANDRA-18618:
-----------------------------------------------
I gave on OWASP and licenses and removed everything related to that from the
PR. Let me leave this ticket address what the title says and move the OWASP
stuff to a separate ticket
https://issues.apache.org/jira/browse/CASSANDRA-18671 - I'll commit the remove
stuff there.
IMHO OWASP deserves republishing a docker image for CircleCI and I don't want
to hold this particular story until that happens. Also the more unrelated
things I want to squizz in a single PR, the more difficult to get agreement
from the all interested parties. My apologize for the confusion, I really want
to finish this one first.
> 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: 7h 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]
