Skip to content
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

[Backport 2.x] Migrating from checkstyle to spotless (#648 and #651) #661

Merged
merged 4 commits into from
Oct 12, 2023

Conversation

VachaShah
Copy link
Collaborator

Description

This is the backport for all spotless changes on main from PRs #648 and #651.

Issues Resolved

List any issues this PR will resolve, e.g. Closes [...].

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

…ect#648)

* Adding spotless for checking and fixing formatting and removing checkstyle

Signed-off-by: Vacha Shah <[email protected]>

* Making build task depend on spotlessJavaCheck

Signed-off-by: Vacha Shah <[email protected]>

* Removing resources and documentation related to checkstyle

Signed-off-by: Vacha Shah <[email protected]>

* Running spotless for json, transport and util folders

Signed-off-by: Vacha Shah <[email protected]>

* Running spotless for samples

Signed-off-by: Vacha Shah <[email protected]>

* Removing commented code

Signed-off-by: Vacha Shah <[email protected]>

---------

Signed-off-by: Vacha Shah <[email protected]>
@dblock dblock merged commit 119cabc into opensearch-project:2.x Oct 12, 2023
25 of 26 checks passed
@VachaShah VachaShah deleted the 2.x branch October 12, 2023 20:46
@BrendonFaleiro BrendonFaleiro mentioned this pull request Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants