You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cassandra.apache.org by er...@apache.org on 2022/10/19 04:25:54 UTC

[cassandra-website] branch trunk updated: DOC - Add doc on checkstyle in CASSANDRA-17876 for contributors

This is an automated email from the ASF dual-hosted git repository.

erickramirezau pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/cassandra-website.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 4e9294444 DOC - Add doc on checkstyle in CASSANDRA-17876 for contributors
4e9294444 is described below

commit 4e92944442f1ca280a42318a159676d33b0cf871
Author: Stefan Miklosovic <sm...@apache.org>
AuthorDate: Mon Sep 26 21:18:41 2022 +0200

    DOC - Add doc on checkstyle in CASSANDRA-17876 for contributors
    
    patch by Stefan Miklosovic; reviewed by Erick Ramirez for CASSANDRA-17924
---
 .../source/modules/ROOT/pages/development/code_style.adoc     | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/site-content/source/modules/ROOT/pages/development/code_style.adoc b/site-content/source/modules/ROOT/pages/development/code_style.adoc
index 3e18e8ebd..2e9e97077 100644
--- a/site-content/source/modules/ROOT/pages/development/code_style.adoc
+++ b/site-content/source/modules/ROOT/pages/development/code_style.adoc
@@ -7,6 +7,17 @@ coding conventions] for anything not expressly outlined in this document.
 
 Note that the project has a variety of styles that have accumulated in different subsystems. Where possible a balance should be struck between these guidelines and the style of the code that is being modified as part of a patch. Patches should also limit their scope to the minimum necessary for safely addressing the concerns of the patch.
 
+== Checkstyle
+
+Cassandra uses https://checkstyle.sourceforge.io/[Checkstyle] project for enforcing various checkstyle policies the project follows. Checkstyle is part of the build from Cassandra 4.1 included. 
+You can consult the checkstyle configuration file called `checkstyle.xml` for the source code in `src` directory and `checkstyle_test.xml` for all code in `test` directory. 
+The configuration files are located in the root of the Cassandra repository. Checkstyle can be executed independently for the main source code as well as for the tests by executing `ant checkstyle` and `ant checkstyle-test` respectively.
+
+The checkstyle target is executed by default when e.g. `build` or `jar` targets are executed. There is a flag you can use for _not_ enforcing checkstyle. This is particularly handy upon development. 
+For example, by default, the checkstyle target checks that your changes in Java code do not include imports which are not used. 
+However, while you develop, you do not want this check to be enforced because you are not interested in it while you develop as your code tends to be in the in-progress state. 
+You can turn whole checkstyle off by specifying `-Dno-checkstyle=true` on the command line, for example like this: `ant build -Dno-checkstyle=true`.
+
 == Naming and Clear Semantics
 
 ==== Class, Method and Variable Naming


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cassandra.apache.org
For additional commands, e-mail: commits-help@cassandra.apache.org