Skip to content

chore: refactored the release.yml file - #1039

Merged
pmathew92 merged 1 commit into
mainfrom
release_pipeline_refactor
Aug 6, 2026
Merged

chore: refactored the release.yml file#1039
pmathew92 merged 1 commit into
mainfrom
release_pipeline_refactor

Conversation

@pmathew92

@pmathew92 pmathew92 commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Changes

This PR refactors the release.yml by removing the java-release.yml file and composing all actions to a single file

Summary by CodeRabbit

  • Release Process

    • Updated the release workflow to publish packages to Maven Central and create GitHub releases directly.
    • Preserved versioning, prerelease checks, release-note retrieval, and duplicate-tag safeguards.
    • Removed the obsolete standalone Java release workflow.
  • Documentation

    • Updated release process documentation to reflect the consolidated workflow and security scanning steps.

@pmathew92
pmathew92 requested a review from a team as a code owner August 6, 2026 03:05
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The reusable Java release workflow was removed. The main release workflow now performs versioning, tag validation, Maven Central publication, and GitHub release creation. Project documentation now references the consolidated workflow.

Changes

Release workflow consolidation

Layer / File(s) Summary
Inline release publication and documentation
.github/workflows/release.yml, .github/workflows/java-release.yml, CLAUDE.md
The reusable Java release job was replaced with inline checkout, release metadata, tag validation, Maven publication, and GitHub release steps. Documentation now references release.yml.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Suggested reviewers: amitsingh05667

Sequence Diagram(s)

sequenceDiagram
  participant ReleaseJob
  participant GitHubRepository
  participant MavenCentral
  participant GitHubRelease
  ReleaseJob->>GitHubRepository: Checkout repository and inspect release tag
  GitHubRepository-->>ReleaseJob: Return source and tag status
  ReleaseJob->>MavenCentral: Publish signed artifacts
  ReleaseJob->>GitHubRelease: Create release with metadata and commit SHA
Loading
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title identifies the refactor of release.yml, which matches the pull request's main change.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch release_pipeline_refactor

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pmathew92 pmathew92 changed the title chore: refacrtored the release.yml file chore: refactored the release.yml file Aug 6, 2026

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
CLAUDE.md (1)

576-586: 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win

Document the automatic GitHub release creation.

The workflow now creates the GitHub release in .github/workflows/release.yml lines 98-106. Line 586 still instructs a manual post-release creation. Update this section to state that the release job validates the tag and creates the GitHub release automatically.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@CLAUDE.md` around lines 576 - 586, Update the “Post-release” section to
remove the manual GitHub release instruction and state that the release job in
release.yml validates the tag and automatically creates the GitHub release,
including the tag and changelog excerpt.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/workflows/release.yml:
- Around line 59-106: Add a top-level description field to the action.yml
metadata for each local action referenced by get_version, get_prerelease,
get_release_notes, tag_exists, maven-publish, and release-create. Use concise,
accurate descriptions of each action’s purpose and ensure all actionlint checks
pass.

---

Outside diff comments:
In `@CLAUDE.md`:
- Around line 576-586: Update the “Post-release” section to remove the manual
GitHub release instruction and state that the release job in release.yml
validates the tag and automatically creates the GitHub release, including the
tag and changelog excerpt.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 9438abb1-4110-409f-a6b1-2bbb578f83ed

📥 Commits

Reviewing files that changed from the base of the PR and between f603638 and ed92189.

📒 Files selected for processing (3)
  • .github/workflows/java-release.yml
  • .github/workflows/release.yml
  • CLAUDE.md
💤 Files with no reviewable changes (1)
  • .github/workflows/java-release.yml

Comment thread .github/workflows/release.yml
@pmathew92
pmathew92 merged commit 1eaa4f3 into main Aug 6, 2026
7 checks passed
@pmathew92
pmathew92 deleted the release_pipeline_refactor branch August 6, 2026 04:59
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.

2 participants