Skip to content

Conversation

@buckett
Copy link
Member

@buckett buckett commented Jan 8, 2026

This switches to a supported method of publishing to the maven repository as the old method is no longer supported.

Copilot AI review requested due to automatic review settings January 8, 2026 10:28
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 updates the Maven publishing configuration by removing profile-based builds and moving plugins directly into the main build section. The change aims to adopt a supported publishing method, but introduces several issues that need to be addressed.

Key Changes:

  • Removed release and sonatype Maven profiles
  • Moved javadoc, source, and GPG signing plugins from profiles to main build configuration
  • Updated CI workflows to remove -Prelease flag from Maven commands

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
pom.xml Removed profile-based plugin configuration and moved javadoc, source, and GPG plugins to main build section
.github/workflows/push.yml Removed -Prelease flag from deploy command
.github/workflows/pr.yml Removed -Prelease flag from package command

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

This switches to a supported method of publishing to the maven repository as the old method is no longer supported.
@buckett buckett merged commit 8bc3280 into master Jan 9, 2026
1 check 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.

1 participant