Skip to content

Updated java ci to avoid unnecessary commits for swagger files#13902

Merged
raulbob merged 1 commit intodevelopmentfrom
task-update_github_java_ci
Apr 7, 2026
Merged

Updated java ci to avoid unnecessary commits for swagger files#13902
raulbob merged 1 commit intodevelopmentfrom
task-update_github_java_ci

Conversation

@raulbob
Copy link
Copy Markdown
Contributor

@raulbob raulbob commented Apr 7, 2026

Add check to avoid committing if no changes are detected.

Summary by CodeRabbit

  • Chores
    • Improved CI/CD pipeline efficiency by implementing conditional commits for automated file updates. The workflow now only commits and pushes when changes are detected, reducing redundant operations and maintaining a cleaner commit history.

Add check to avoid committing if no changes are detected.
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 7, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5d173dc2-60b6-48c2-b781-61fa050849cc

📥 Commits

Reviewing files that changed from the base of the PR and between c56cc1f and c8b1f59.

📒 Files selected for processing (1)
  • .github/workflows/ci.yml

📝 Walkthrough

Walkthrough

The CI workflow step for updating Swagger documentation on the development branch now conditionally commits only when there are staged changes. It checks if staged diffs exist before executing commit and push operations, avoiding redundant commits when files are already synchronized.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/ci.yml
Added conditional logic using git diff --cached --quiet to skip commit/push operations when Swagger files have no staged changes, with informational output when files are already up to date.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

Poem

🐰 A rabbit's ode to cleaner workflows:

No needless commits to clutter the way,
Git checks before pushing—hooray, hooray!
When Swagger's unchanged, we skip with a bound,
Efficiency gains without a sound. 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive The description is incomplete. It lacks the required issue reference (Fixes #) specified in the repository's description template. Add the issue number reference using 'Fixes #ISSUE_NUMBER' format to match the repository's pull request template requirements.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding conditional logic to avoid unnecessary commits for swagger files in the CI workflow.
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
  • Commit unit tests in branch task-update_github_java_ci

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.

@raulbob raulbob merged commit 1831a06 into development Apr 7, 2026
7 checks passed
@raulbob raulbob deleted the task-update_github_java_ci branch April 7, 2026 11:57
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