Skip to content

Conversation

tcheeric
Copy link
Owner

@tcheeric tcheeric commented Aug 9, 2025

Summary

  • publish releases to GitHub Packages via distributionManagement
  • release workflow authenticates to GitHub Packages and deploys artifacts

Testing

  • ./mvnw -q verify (fails: java.net.SocketException: Network is unreachable)

Network Access

  • repo.maven.apache.org – network is unreachable

https://chatgpt.com/codex/tasks/task_b_6896c2a8ef908331a8f42dcaec869f57

Copy link
Contributor

@Copilot 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 configures GitHub Packages as an additional deployment target for Maven artifacts alongside the existing OSSRH deployment. The changes enable the project to publish releases to both Maven Central and GitHub Packages.

  • Adds GitHub Packages repository configuration to Maven distributionManagement
  • Updates the release workflow to authenticate with GitHub Packages using GITHUB_TOKEN
  • Introduces a separate deployment step for GitHub Packages using altDeploymentRepository

Reviewed Changes

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

File Description
pom.xml Adds GitHub Packages repository configuration to distributionManagement
.github/workflows/release.yml Configures GitHub authentication and adds deployment step for GitHub Packages

@@ -18,6 +18,9 @@ jobs:
distribution: 'temurin'
java-version: '21'
cache: 'maven'
server-id: github
server-username: ${{ github.actor }}
Copy link
Preview

Copilot AI Aug 9, 2025

Choose a reason for hiding this comment

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

[nitpick] The server-username should be configured as an environment variable (GITHUB_ACTOR) instead of using ${{ github.actor }} directly in the server configuration. This would be more consistent with the server-password configuration pattern.

Copilot uses AI. Check for mistakes.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tcheeric tcheeric merged commit 700ce78 into develop Aug 9, 2025
1 check failed
@tcheeric tcheeric deleted the codex/configure-maven-for-github-packages branch August 9, 2025 03:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant