Skip to content

Update build.gradle URLs#175

Merged
isaacbrodsky merged 2 commits intomasterfrom
isaacbrodsky-patch-1
Jul 20, 2025
Merged

Update build.gradle URLs#175
isaacbrodsky merged 2 commits intomasterfrom
isaacbrodsky-patch-1

Conversation

@isaacbrodsky
Copy link
Copy Markdown
Collaborator

@isaacbrodsky isaacbrodsky commented Jul 18, 2025

I am not 100% sure about these URLs because it's not my token, but I understand ossrh is EOL and this is the replacement, so it seems like the next thing to do: https://central.sonatype.org/pages/ossrh-eol/#central-support

@coveralls
Copy link
Copy Markdown

coveralls commented Jul 18, 2025

Pull Request Test Coverage Report for Build #526

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 94.437%

Totals Coverage Status
Change from base Build #521: 0.0%
Covered Lines: 523
Relevant Lines: 546

💛 - Coveralls

Copy link
Copy Markdown
Collaborator

@dannyiland dannyiland left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is https://ossrh-staging-api.central.sonatype.com/service/local/ the correct URL or do you want non staging?

@isaacbrodsky
Copy link
Copy Markdown
Collaborator Author

Is https://ossrh-staging-api.central.sonatype.com/service/local/ the correct URL or do you want non staging?

According to https://central.sonatype.org/publish/publish-portal-ossrh-staging-api/#configuring-the-repository this is the replacement URL, but to be honest I am not 100% sure.

https://ossrh-api.central.sonatype.com/service/local/ does not resolve for me. I think "staging" here doesn't mean a staging copy of the service but rather that it is staging the artifact before release (but again I am guessing)

Copy link
Copy Markdown
Collaborator

@dfellis dfellis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We won't know until we try, so let's try it.

@isaacbrodsky isaacbrodsky merged commit 52eb3e2 into master Jul 20, 2025
82 checks passed
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.

5 participants