Skip to content

Conversation

tcheeric
Copy link
Owner

Summary

  • split create-release and asset upload steps in release workflow
  • limit GH_TOKEN usage to CLI steps

Testing

  • mvn -q verify (fails: Could not find a valid Docker environment)
  • actionlint .github/workflows/release.yml

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

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 splits the GitHub release workflow by separating the release creation and asset upload steps, removing redundant release creation logic and limiting the scope of GH_TOKEN usage to only the CLI steps that require it.

  • Removes duplicate release creation code that was using GitHub CLI
  • Streamlines the workflow by relying on the existing GitHub action for release creation
  • Improves security by limiting GH_TOKEN scope to only necessary steps

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

@@ -59,14 +59,6 @@ jobs:
tag_name: ${{ github.ref }}
release_name: ${{ github.ref_name }}
Copy link
Preview

Copilot AI Aug 12, 2025

Choose a reason for hiding this comment

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

The removed CLI command used --title "${{ github.ref }}" but the action uses release_name: ${{ github.ref_name }}. This creates an inconsistency where the release title will now be the tag name without the 'refs/tags/' prefix, potentially changing the expected release naming format.

Suggested change
release_name: ${{ github.ref_name }}
release_name: ${{ github.ref }}

Copilot uses AI. Check for mistakes.

@codecov-commenter
Copy link

⚠️ 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.

📢 Thoughts on this report? Let us know!

@tcheeric tcheeric merged commit 0cbfb61 into main Aug 12, 2025
3 checks passed
@tcheeric tcheeric deleted the codex/split-and-refine-github-release-workflow branch August 12, 2025 21:59
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.

2 participants