Skip to content

Commit 9a46978

Browse files
ci(coverage): drop --depth=1 on the base fetch — fix diff-cover 'no merge base' (#276)
The patch-coverage gate did 'git fetch origin <base> --depth=1', truncating the base to one commit with no common ancestor with the PR branch whenever the branch was behind base — so diff-cover crashed with 'fatal: origin/<base>...HEAD: no merge base' and reded coverage on PRs that were merely stale (hit repeatedly). The checkout is already fetch-depth:0, so a plain 'git fetch origin <base>' resolves the merge-base reliably. Co-authored-by: Claude <noreply@anthropic.com>
1 parent 426e5a9 commit 9a46978

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

.github/workflows/coverage.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,9 @@ jobs:
176176
if: github.event_name == 'pull_request'
177177
working-directory: api
178178
run: |
179-
git fetch origin "${{ github.base_ref }}" --depth=1 || true
179+
# Full base history (NOT --depth=1): a shallow base fetch shares no merge-base
180+
# with a PR branch that is behind base → diff-cover "no merge base" crash.
181+
git fetch origin "${{ github.base_ref }}" || true
180182
diff-cover coverage.xml \
181183
--compare-branch="origin/${{ github.base_ref }}" \
182184
--fail-under=100

0 commit comments

Comments
 (0)