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

Removing jcenter repo from repositories blocks in gradle files #261

Merged

Conversation

martin-gaievski
Copy link
Member

Signed-off-by: Martin Gaievski gaievski@amazon.com

Description

Removing all direct dependencies on jcenter() within a repositories block. Repository has been shut down and replaced by mavenCentral(), see parent issue for detailed explanation.

Issues Resolved

Remove jcenter repository #260

Check List

  • Commits are signed as per the DCO using --signoff

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.

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
@martin-gaievski martin-gaievski self-assigned this Jan 13, 2022
@martin-gaievski martin-gaievski added v1.3.0 Issues and PRs related to version 1.3.0 bug Something isn't working labels Jan 13, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #261 (d0ce8db) into main (e781ad3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##               main     #261   +/-   ##
=========================================
  Coverage     83.32%   83.32%           
  Complexity      883      883           
=========================================
  Files           127      127           
  Lines          3832     3832           
  Branches        361      361           
=========================================
  Hits           3193     3193           
  Misses          477      477           
  Partials        162      162           

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 e781ad3...d0ce8db. Read the comment docs.

@martin-gaievski martin-gaievski marked this pull request as ready for review January 13, 2022 19:41
@martin-gaievski martin-gaievski requested a review from a team January 13, 2022 19:41
@jmazanec15 jmazanec15 added the backport 1.2 label to add to PRs to auto backport label Jan 13, 2022
@martin-gaievski martin-gaievski merged commit 7c8c3e7 into opensearch-project:main Jan 13, 2022
martin-gaievski added a commit to martin-gaievski/k-NN that referenced this pull request Jan 13, 2022
…ject#261)

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
martin-gaievski added a commit that referenced this pull request Jan 13, 2022
Signed-off-by: Martin Gaievski <gaievski@amazon.com>
martin-gaievski added a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
…ject#261)

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
martin-gaievski added a commit to martin-gaievski/k-NN that referenced this pull request Mar 7, 2022
…ject#261)

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
martin-gaievski added a commit to martin-gaievski/k-NN that referenced this pull request Mar 30, 2022
…ject#261)

Signed-off-by: Martin Gaievski <gaievski@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.2 label to add to PRs to auto backport bug Something isn't working v1.3.0 Issues and PRs related to version 1.3.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants