Skip to content

Comments

ci: simplify release workflow publish command#134

Merged
nobodyiam merged 1 commit intoapolloconfig:mainfrom
nobodyiam:codex/central-plugin-release
Feb 19, 2026
Merged

ci: simplify release workflow publish command#134
nobodyiam merged 1 commit intoapolloconfig:mainfrom
nobodyiam:codex/central-plugin-release

Conversation

@nobodyiam
Copy link
Member

@nobodyiam nobodyiam commented Feb 19, 2026

What's the purpose of this PR

Simplify the Sonatype Central release workflow command by removing redundant shell wrapper settings and keeping the publish step minimal.

Which issue(s) this PR fixes:

Fixes #N/A (release workflow cleanup)

Brief changelog

  • simplify release workflow run command to a single-line Maven publish invocation
  • remove unnecessary set -eo pipefail wrapper from the publish step

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Read the Contributing Guide before making this pull request.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit tests to verify the code.
  • Run mvn clean test to make sure this pull request doesn't break anything.
  • Update the CHANGES log.

Summary by CodeRabbit

  • Chores
    • Optimized the release deployment workflow process.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 19, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

The Publish to Sonatype Central step in the release workflow was simplified from a multi-line shell script with explicit error handling (set -eo pipefail) to a single mvn clean deploy command, reducing the deployment logic from three lines to one.

Changes

Cohort / File(s) Summary
CI/CD Workflow
.github/workflows/release.yml
Simplified the Publish to Sonatype Central step by consolidating the multi-line shell script into a single mvn clean deploy command; removed explicit set -eo pipefail error handling directive.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • ci: externalize release workflow helper scripts #128: Modified the same publish step in .github/workflows/release.yml, previously introducing set -eo pipefail and tee for release script handling, making this change a reversal/simplification of that earlier approach.

Poem

🐰 The pipeline hops with grace so lean,
One command cleans what once was seen,
Multi-lines become just one,
Deploy the release, the job is done! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: simplify release workflow publish command' directly aligns with the main change: simplifying the Sonatype Central release workflow by removing redundant shell settings and consolidating the Maven command into a single invocation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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 and usage tips.

@codecov
Copy link

codecov bot commented Feb 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.01%. Comparing base (d4b76f8) to head (790649f).
⚠️ Report is 24 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main     #134      +/-   ##
============================================
+ Coverage     68.68%   71.01%   +2.32%     
- Complexity     1503     1634     +131     
============================================
  Files           212      224      +12     
  Lines          6396     6727     +331     
  Branches        647      679      +32     
============================================
+ Hits           4393     4777     +384     
+ Misses         1673     1600      -73     
- Partials        330      350      +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@nobodyiam nobodyiam merged commit 5a3fec9 into apolloconfig:main Feb 19, 2026
16 checks passed
@nobodyiam nobodyiam deleted the codex/central-plugin-release branch February 19, 2026 08:51
@github-actions github-actions bot locked and limited conversation to collaborators Feb 19, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant