You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@solr.apache.org by ds...@apache.org on 2023/03/17 06:05:12 UTC

[solr] branch FAQ-googleJavaFormat created (now 78e1512c177)

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

dsmiley pushed a change to branch FAQ-googleJavaFormat
in repository https://gitbox.apache.org/repos/asf/solr.git


      at 78e1512c177 Update FAQ.adoc: Google Java Format

This branch includes the following new commits:

     new 78e1512c177 Update FAQ.adoc: Google Java Format

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.



[solr] 01/01: Update FAQ.adoc: Google Java Format

Posted by ds...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

dsmiley pushed a commit to branch FAQ-googleJavaFormat
in repository https://gitbox.apache.org/repos/asf/solr.git

commit 78e1512c1771d0c630fa0e3702825bde50704c60
Author: David Smiley <ds...@salesforce.com>
AuthorDate: Fri Mar 17 02:05:06 2023 -0400

    Update FAQ.adoc: Google Java Format
---
 dev-docs/FAQ.adoc | 23 ++++++++++++++---------
 1 file changed, 14 insertions(+), 9 deletions(-)

diff --git a/dev-docs/FAQ.adoc b/dev-docs/FAQ.adoc
index 267b137bad1..8974560b792 100644
--- a/dev-docs/FAQ.adoc
+++ b/dev-docs/FAQ.adoc
@@ -30,20 +30,25 @@ via `bin/solr start -e cloud -noprompt` and then as you make changes to assets i
 run `gradle dev` to redeploy the web assets. Do a  hard refresh in your browser
 to pick up your changes.
 
-=== How do we ensure coding standards and quality?
+=== Are there Coding Standards to follow?
 
-We use a number of tools for ensuring that Solr's codebase follows our community standards.
-The most important tool is the very rich testing infrastructure that Solr enjoys.
-We strive for testing of every aspect of Solr.
-If you find a bug, write a test demonstrating it, so we can ensure that once the bug is squashed, it stays squashed.
-
-Beyond testing, we also have some tools for ensuring coding standards are followed.
-https://github.com/diffplug/spotless[Spotless] is used to format the Java source code so that everyone uses the same formatting patterns.
-Run Spotless via:
+Please follow the https://google.github.io/styleguide/javaguide.html[Google Java Style Guide].
+We don't follow all aspects strictly; much code was written before this style guide was selected.
+The whitespace formatting aspect of this guide is mandatory and strictly enforced with tooling.
+To format the code automatically using "Spotless", do the following:
 
 `./gradlew tidy`
 
 Learn more via `./gradlew :helpFormatting`.
+Major IDE vendors have plugins for this style guide to help keep your code formatted as you work:
+
+ * https://github.com/google/google-java-format#eclipse
+ * https://plugins.jetbrains.com/plugin/8527-google-java-format
+
+We use a number of tools for ensuring that Solr's codebase follows our community standards.
+The most important tool is the very rich testing infrastructure that Solr enjoys.
+We strive for testing of every aspect of Solr.
+If you find a bug, write a test demonstrating it, so we can ensure that once the bug is squashed, it stays squashed.
 
 The Eclipse IDE's linting tool ECJ is applied as part of the `check` task.