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] The spotless should take care of unified imports ordering (test/ subprojects) (#9351) #9370

Merged

Conversation

reta
Copy link
Collaborator

@reta reta commented Aug 15, 2023

Backport of #9351 to 2.x

…rojects) (opensearch-project#9351)

Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>
(cherry picked from commit a1faafe)
@opensearch-trigger-bot
Copy link
Contributor

Compatibility status:



> Task :checkCompatibility
Incompatible components: [https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/performance-analyzer.git]
Components skipped due to git failures: [https://github.com/opensearch-project/opensearch-oci-object-storage.git]
Compatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/performance-analyzer-rca.git]

BUILD SUCCESSFUL in 25m 42s

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Aug 15, 2023

Codecov Report

Merging #9370 (a643c89) into 2.x (cdf6e67) will decrease coverage by 0.23%.
Report is 3 commits behind head on 2.x.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.x    #9370      +/-   ##
============================================
- Coverage     70.92%   70.69%   -0.23%     
+ Complexity    57593    57455     -138     
============================================
  Files          4761     4761              
  Lines        271842   271842              
  Branches      40086    40086              
============================================
- Hits         192793   192190     -603     
- Misses        62554    63151     +597     
- Partials      16495    16501       +6     
Files Changed Coverage Δ
...h/client/benchmark/ops/bulk/BulkBenchmarkTask.java 0.00% <ø> (ø)
...in/java/org/opensearch/plugin/noop/NoopPlugin.java 0.00% <ø> (ø)
...ch/plugin/noop/action/bulk/RestNoopBulkAction.java 0.00% <ø> (ø)
...ugin/noop/action/bulk/TransportNoopBulkAction.java 0.00% <ø> (ø)
.../noop/action/search/TransportNoopSearchAction.java 0.00% <ø> (ø)
...main/java/org/opensearch/client/ClusterClient.java 8.57% <ø> (ø)
...java/org/opensearch/client/GetAliasesResponse.java 88.75% <ø> (ø)
...main/java/org/opensearch/client/IndicesClient.java 1.77% <ø> (ø)
...rg/opensearch/client/IndicesRequestConverters.java 71.03% <ø> (+0.55%) ⬆️
.../main/java/org/opensearch/client/IngestClient.java 15.78% <ø> (ø)
... and 113 more

... and 481 files with indirect coverage changes

@kotwanikunal kotwanikunal merged commit df0b68f into opensearch-project:2.x Aug 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants