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 1.x] [CODE STYLE] Switch checkstyle to spotless #298

Merged
merged 2 commits into from
Feb 18, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 0db9b2e from #297

Switches checkstyle functionality to spotless. Sets ratchetFrom to
origin/main. This will mean it will only fail on files recently changed.
Pull formatting from OpenSearch.

Signed-off-by: John Mazanec <[email protected]>
(cherry picked from commit 0db9b2e)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team February 18, 2022 17:41
Signed-off-by: John Mazanec <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Feb 18, 2022

Codecov Report

Merging #298 (0db9b2e) into 1.x (1995c95) will not change coverage.
The diff coverage is 100.00%.

❗ Current head 0db9b2e differs from pull request most recent head 95c4b00. Consider uploading reports for the commit 95c4b00 to get more accurate results

Impacted file tree graph

@@            Coverage Diff            @@
##                1.x     #298   +/-   ##
=========================================
  Coverage     83.38%   83.38%           
  Complexity      885      885           
=========================================
  Files           127      127           
  Lines          3834     3834           
  Branches        361      361           
=========================================
  Hits           3197     3197           
  Misses          475      475           
  Partials        162      162           
Impacted Files Coverage Δ
.../main/java/org/opensearch/knn/index/IndexUtil.java 56.66% <100.00%> (ø)
...n/java/org/opensearch/knn/index/KNNIndexShard.java 93.02% <100.00%> (ø)
.../main/java/org/opensearch/knn/index/KNNWeight.java 75.34% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1995c95...95c4b00. Read the comment docs.

@jmazanec15 jmazanec15 merged commit 3ff3077 into 1.x Feb 18, 2022
@github-actions github-actions bot deleted the backport/backport-297-to-1.x branch February 18, 2022 18:46
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
…-project#298)

Switches checkstyle functionality to spotless. Sets ratchetFrom to
origin/1.x. This will mean it will only fail on files recently changed.
Pull formatting from OpenSearch.

Signed-off-by: John Mazanec <[email protected]>
(cherry picked from commit 0db9b2e)
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
…-project#298)

Switches checkstyle functionality to spotless. Sets ratchetFrom to
origin/1.x. This will mean it will only fail on files recently changed.
Pull formatting from OpenSearch.

Signed-off-by: John Mazanec <[email protected]>
(cherry picked from commit 0db9b2e)
Signed-off-by: Martin Gaievski <[email protected]>
martin-gaievski pushed a commit to martin-gaievski/k-NN that referenced this pull request Mar 31, 2022
…-project#298)

Switches checkstyle functionality to spotless. Sets ratchetFrom to
origin/1.x. This will mean it will only fail on files recently changed.
Pull formatting from OpenSearch.

Signed-off-by: John Mazanec <[email protected]>
(cherry picked from commit 0db9b2e)
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.

3 participants