Skip to content

Conversation

@tmccombs
Copy link
Contributor

This simplifies the release process, and updates to work with the new "Central Portal" for Maven Central.

Hopefully this fixes publishing the package ot maven central.

This simplifies the release process, and updates to work with the new "Central Portal" for Maven
Central.

Hopefully this fixes publishing the package ot maven central.
@tmccombs tmccombs requested review from Copilot and sriraamas October 14, 2025 18:50
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the Maven Central publishing workflow by migrating from manual PGP and Sonatype plugins to the automated sbt-ci-release plugin, which supports the new Central Portal publishing method.

Key changes:

  • Replaces sbt-pgp and sbt-sonatype plugins with sbt-ci-release
  • Updates sbt version from 1.10.7 to 1.11.7
  • Simplifies CI workflow by replacing custom publishing script with single ci-release command

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated no comments.

File Description
project/build.sbt Consolidates publishing plugins by replacing sbt-pgp and sbt-sonatype with sbt-ci-release
project/build.properties Updates sbt to version 1.11.7 to support the new plugin
admin/build.sbt Removes custom Sonatype publishing configuration now handled by sbt-ci-release
.github/workflows/ci.yml Simplifies release workflow by replacing manual PGP import and conditional publishing logic with automated ci-release command

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@aegbert5 aegbert5 left a comment

Choose a reason for hiding this comment

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

Changes make sense to me. Is there a way to test these changes before merging? Or do we need to merge first?

@tmccombs
Copy link
Contributor Author

I can test that the normal build works (as you can see by the passing checks), but unfortunately, there isn't really a good way to test the deployment part without merging.

@tmccombs tmccombs merged commit 1aecc7b into master Oct 15, 2025
1 check passed
@tmccombs tmccombs deleted the sbt-ci-release branch October 15, 2025 23:34
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