Skip to content

fix: fetch full git history in CI checkout#6

Merged
RishabhS7 merged 1 commit into
masterfrom
fix/git
Feb 5, 2026
Merged

fix: fetch full git history in CI checkout#6
RishabhS7 merged 1 commit into
masterfrom
fix/git

Conversation

@rongquan1
Copy link
Copy Markdown

@rongquan1 rongquan1 commented Feb 5, 2026

Summary by CodeRabbit

  • Chores
    • Updated CI workflow configuration to ensure complete commit history is available during automated builds.

@rongquan1 rongquan1 requested a review from RishabhS7 February 5, 2026 09:12
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

Modified the GitHub Actions CI workflow to fetch complete git history instead of a shallow clone by adding fetch-depth: 0 to the checkout step. This configuration change enables access to the full commit history during workflow execution.

Changes

Cohort / File(s) Summary
CI Workflow Configuration
.github/workflows/ci.yml
Added fetch-depth: 0 parameter to checkout action to fetch complete git history instead of shallow clone.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Full history now flows like a stream so wide,
No shallow clones to hide what's inside,
The workflow runs deep, with git's truth unfurled,
Complete context brought to our CI world! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request description is missing entirely. The template requires Summary, Changes, and Issues sections, none of which are provided. Add a complete description following the template with background information, detailed list of changes, and related issues or stories.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding fetch-depth to enable full git history fetching in the CI checkout step.
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
  • Commit unit tests in branch fix/git

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.

@RishabhS7 RishabhS7 merged commit e48e616 into master Feb 5, 2026
5 checks passed
@RishabhS7 RishabhS7 deleted the fix/git branch February 5, 2026 09:22
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