Repository navigation
Give CodeQL a full clone so Nerdbank.GitVersioning can version the build - #32
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
|
|
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
The pull request correctly implements a full git clone by setting fetch-depth: 0 in the CodeQL workflow. This change is essential for Nerdbank.GitVersioning to accurately calculate version height during the C# 'Autobuild' step.
According to Codacy analysis, the changes are up to standards, and no new quality issues or security vulnerabilities were introduced. The review identifies no blockers to merging, though verification of the CI job success is recommended as a final step.
Test suggestions
- Verify that the CodeQL 'Analyze' job for C# completes the 'Autobuild' step successfully without version calculation errors.
Prompt proposal for missing tests
Consider implementing these tests if applicable:
1. Verify that the CodeQL 'Analyze' job for C# completes the 'Autobuild' step successfully without version calculation 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