Details
-
Epic
-
Status: Triage Needed
-
Normal
-
Resolution: Unresolved
-
None
-
None
-
None
-
Code Style & Static Analysis
-
All
-
None
Description
This is an umbrella ticket that aggregates all activities related to improving code style guides as well as adding new tools for code analysis.
Despite the fact that expanding the code style rules and adding such checks to the source code may take a long time, the main goals for this epic are:
- add a static code analysis automatic checks support (SpotBug) to the build;
- in terms of automatic code checking make GitHub pull requests more convenient;
- update the checkstyle configuration;
Cassandra code style guide:
https://cassandra.apache.org/_/development/code_style.html
https://cwiki.apache.org/confluence/display/CASSANDRA2/CodeStyle
Discussion Code style and static analysis on the dev list:
https://lists.apache.org/thread/11j0hrv2bkx60xk7zvlgqgjwo982qv6h
Discussion Add SpotBugs to the build on the dev list:
https://lists.apache.org/thread/1ro1mvkpvt4vr24nw7dbpdlxo82mq3hz
Attachments
Attachments
Issues in epic
CASSANDRA-9209 | Add static analysis to report any AutoCloseable objects that are not encapsulated in a try/finally block | Open | Unassigned | |||
CASSANDRA-9919 | Cleanup closeable iterator usage | Open | Unassigned | |||
CASSANDRA-17124 | Enable full checkstyle on tests | Open | Unassigned | |||
CASSANDRA-17400 | Fail build or warn when closeable reference is not closed in tests | Triage Needed | Unassigned | |||
CASSANDRA-17925 | Java source code should have sorted imports as defined in the codestyle | Patch Available | Maxim Muzafarov | |||
CASSANDRA-18077 | Add SpotBugs to the build | Open | Unassigned | |||
|
CASSANDRA-18089 | The source code must obey the avoid star import checkstyle rule | Resolved | Maxim Muzafarov | ||
CASSANDRA-18239 | Replace eclipse warnings based static code analysis with something better (CheckerFramework) | Review In Progress | Jacek Lewandowski | |||
CASSANDRA-18277 | Move the IntelliJ Idea code style and inspections configuration to the project's root .idea directory | Open | Maxim Muzafarov | |||
CASSANDRA-18390 | Run Sonar analyzer over the Cassandra project | Patch Available | Maxim Muzafarov | |||
|
CASSANDRA-18785 | Add optional task to run Sonar analysis | Resolved | Jacek Lewandowski | ||
|
CASSANDRA-18853 | IDEA to mark unused imports as error | Resolved | Berenguer Blasi | ||
|
CASSANDRA-18865 | Enable unnecessary import from the same package check in IDEs | Resolved | Maxim Muzafarov |