Repository navigation
Give CodeQL a full clone so Nerdbank.GitVersioning can version the build - #39
Conversation
…rsion the build CodeQL's autobuild failed on every run because the default shallow checkout lacks the history Nerdbank.GitVersioning needs to compute the version height. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Coverage variation | ✅ +0.00% coverage variation (-1.00%) |
| Diff coverage | ✅ ∅ diff coverage |
Coverage variation details
Coverable lines Covered lines Coverage Common ancestor commit (994a607) 193 134 69.43% Head commit (774c0ae) 193 (+0) 134 (+0) 69.43% (+0.00%) Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch:
<coverage of head commit> - <coverage of common ancestor commit>
Diff coverage details
Coverable lines Covered lines Diff coverage Pull request (#39) 0 0 ∅ (not applicable) Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified:
<covered lines added or modified>/<coverable lines added or modified> * 100%
AI Reviewer: first review requested successfully. AI can make mistakes. Always validate suggestions.
TIP This summary will be updated as you push new changes.
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
There was a problem hiding this comment.
Pull Request Overview
This PR correctly configures the CodeQL workflow to perform a full Git clone. By setting 'fetch-depth: 0' in the 'actions/checkout' step, 'Nerdbank.GitVersioning' will have access to the full commit history required to calculate version heights during the C# autobuild process. This addresses the 'Shallow clone lacks the objects required' error typically encountered in this environment. The changes meet all acceptance criteria, and Codacy reports that the PR is up to standards with no new issues identified.
Test suggestions
- Successful execution of the CodeQL 'Autobuild' step for C# without 'Shallow clone lacks the objects required' errors.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Successful execution of the CodeQL 'Autobuild' step for C# without 'Shallow clone lacks the objects required' errors.
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
Cause
The CodeQL
Analyze (csharp)job fails onmainand on pull requests.actions/checkoutdefaults to a shallow clone (fetch-depth: 1), so Nerdbank.GitVersioning cannot compute the version height and autobuild stops with:Fix
Set
fetch-depth: 0on the checkout step of the CodeQL workflow so the job gets full history. This is the same one-line change proven on panoramicdata/HubSpot.Api#8.Tracked under OPS-157454.
🤖 Generated with Claude Code