Bump spotless to 8.0.0

[Pankraz76]

[Describe what this change achieves]

while experimenting with rewrite i came across this custom solution for what recently has been integrated into spot.

• activate removeWildcardImports() diffplug/spotless#2526
• removeUnusedPrivateMethods Pankraz76/OpenSearch#2

could not do it without @iddeepak, thanks for the decent impl.

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

• Functionality includes testing.
• API changes companion pull request created, if applicable.
• Public documentation issue/PR created, if applicable.

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.

[github-actions]

❌ Gradle check result for 2ea7836: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[Pankraz76]

closing due to conflicts. Need to merge this close by due to the nature of format.

Its trivial anyways. The actual cause here is to check why CI is failing to bump this.

[Pankraz76]

assuming ready to go:

• Fix doc for ForbidWildcardImportsStep diffplug/spotless#2668

[github-actions]

✅ Gradle check result for f95f1d9: SUCCESS

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@6b5c08a). Learn more about missing BASE report.
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #18770   +/-   ##
=======================================
  Coverage        ?   73.17%           
  Complexity      ?    71191           
=======================================
  Files           ?     5766           
  Lines           ?   325467           
  Branches        ?    47084           
=======================================
  Hits            ?   238152           
  Misses          ?    68162           
  Partials        ?    19153           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[sandeshkr419]

@Pankraz76 Were we waiting on diffplug/spotless#2571 which I see is part of 8.0 release.

Probably good to upgrade spotless to 8.0 now?

[Pankraz76]

@Pankraz76 Were we waiting on diffplug/spotless#2571 which I see is part of 8.0 release.

Probably good to upgrade spotless to 8.0 now?

Thanks for catching up on this as well, together we can automate and improve this codebase sustaniable.

[github-actions]

❌ Gradle check result for 19c1a12: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[Pankraz76]

might be IO:

  1> org.opensearch.transport.ConnectTransportException: [node_t0][127.0.0.1:38991] connect_exception

[Pankraz76]

rebase

[github-actions]

❌ Gradle check result for e137f0a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

[sandeshkr419]

* What went wrong:
Execution failed for task ':modules:aggs-matrix-stats:spotlessJava'.
> Cannot fingerprint input property 'stepsInternalEquality': value 'com.diffplug.spotless.ConfigurationCacheHackList@2154da26' cannot be serialized.

Seems like intermittent as it succeeded on my machine with this change checked out. Re-running gradle check.

[github-actions]

✅ Gradle check result for e137f0a: SUCCESS