-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Include Spotless Gradle precommit hook #104
Comments
Gradle offers this documentation that could be useful for the implementation of the generic For now we will work on a basic implementation of the functionality, so this improvement will be worked on later. |
The spotless tool is applied and working correctly with the The problem with the hook is that it seems to require Once the pre-commit hook is working successfully, I will apply the required changes to the other plugin. |
Spotless was implemented altogether with a git pre-commit hook, also a root Gradle project was initialized to handle generic actions for all the plugins (subprojects) |
Description
OpenSearch's codebase, and by inheritance Wazuh Indexer's, uses a precommit hook to format and lint the Java code using Spotless. In order to guarantee that our plugin's code is consistent and well maintained, we will include this precommit hook to our plugin's
build.gradle
.The code conventions to be used are the ones defined in the Developer Guide. We will inherit the code and naming conventions from OpenSearch to keep all the codebase consistent.
Explore if we can add a root Grade project to simplify the configuration. The subprojects (each plugin's Gradle project), will inherit the precommit hook from the root project.
Functional requirements
Implementation restrictions
gradle/formatting.gradle
).license-header.txt
).com.wazuh
is processed.* Optional
Resources
The text was updated successfully, but these errors were encountered: