Skip to content

Attempt to fix maven deploy using Maven Central#1582

Merged
bodiam merged 1 commit intomainfrom
attempt-to-fix-deploy
Jul 7, 2025
Merged

Attempt to fix maven deploy using Maven Central#1582
bodiam merged 1 commit intomainfrom
attempt-to-fix-deploy

Conversation

@bodiam
Copy link
Copy Markdown
Contributor

@bodiam bodiam commented Jul 6, 2025

No description provided.

@bodiam bodiam changed the title Attempt to fix maven deploy. Attempt to fix maven deploy using Maven Central Jul 6, 2025
@what-the-diff
Copy link
Copy Markdown

what-the-diff bot commented Jul 6, 2025

PR Summary

  • Shifted From OSSRH to Maven Central in GitHub Actions Workflows
    The team updated our project's GitHub Actions workflows. We used to use OSSRH (Open Source Software Repository Hosting) as our primary Maven server, but we're now using Maven Central. This implies that our project's build artifacts are now published to Maven Central, which is a more commonly used and recognized repository in the Java ecosystem.

  • Updated Server ID and Environment Variables in Workflows
    According to the changes mentioned above, the server-id has been updated from ossrh to central in our workflows (i.e., deploy.yml and release-from-tag.yml). We've also updated environment variables for Maven username and password to align with this change.

  • Removed 'distributionManagement' Section in 'pom.xml'
    Previously, our project pom.xml file contained a distributionManagement section that pointed to OSSRH repositories for distributing software artifacts. After this update, that section has been removed. This change is part of the shift from OSSRH to Maven Central.

  • New Maven Plugin for Publishing To Maven Central
    The pom.xml file now contains a new plugin (central-publishing-maven-plugin). This plugin will facilitate the publishing of our project's software artifacts to Maven Central.

@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.43%. Comparing base (9cd0dfd) to head (feb2564).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1582      +/-   ##
============================================
+ Coverage     92.35%   92.43%   +0.07%     
- Complexity     3354     3357       +3     
============================================
  Files           331      331              
  Lines          6620     6620              
  Branches        655      655              
============================================
+ Hits           6114     6119       +5     
+ Misses          347      344       -3     
+ Partials        159      157       -2     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Collaborator

@kingthorin kingthorin left a comment

Choose a reason for hiding this comment

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

I have no idea what else might be needed but this seems reasonable

@bodiam
Copy link
Copy Markdown
Contributor Author

bodiam commented Jul 7, 2025

@kingthorin I have really no idea what I'm doing here myself, just experimenting to see how far I get.

@bodiam bodiam merged commit 92710ed into main Jul 7, 2025
13 checks passed
@bodiam bodiam deleted the attempt-to-fix-deploy branch July 7, 2025 11:19
@kingthorin
Copy link
Copy Markdown
Collaborator

Are you going to remove 2.4.4 and try again now that this is in place?

@asolntsev asolntsev added this to the 2.4.4 milestone Jul 14, 2025
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.

4 participants