Code Quality Analysis #18
code-review.yml
on: workflow_dispatch
Code Quality Analysis
5m 12s
Annotations
12 warnings
|
Code Quality Analysis
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/checkout@v4, actions/setup-dotnet@v4, github/codeql-action/analyze@v3, github/codeql-action/init@v3. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Git/GitProvider.cs#L273
Argument of type 'List<Scope?>' cannot be used for parameter 'mergeFromScopes' of type 'IReadOnlyList<Scope>' in 'void GitProvider.UpdateScopeMergeParents(Differences deltas, Scope mergeIntoScope, IReadOnlyList<Scope> mergeFromScopes, Dictionary<string, string> deleted, string commitHash)' due to differences in the nullability of reference types.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Git/GitProvider.cs#L269
Dereference of a possibly null reference.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L40
Non-nullable property 'FromServerPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L38
Non-nullable property 'ServerPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L33
Non-nullable property 'LocalPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Git/GitProvider.cs#L273
Argument of type 'List<Scope?>' cannot be used for parameter 'mergeFromScopes' of type 'IReadOnlyList<Scope>' in 'void GitProvider.UpdateScopeMergeParents(Differences deltas, Scope mergeIntoScope, IReadOnlyList<Scope> mergeFromScopes, Dictionary<string, string> deleted, string commitHash)' due to differences in the nullability of reference types.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Git/GitProvider.cs#L269
Dereference of a possibly null reference.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L40
Non-nullable property 'FromServerPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L38
Non-nullable property 'ServerPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis:
CSharpCodeAnalyst.History/Model/ChangeItem.cs#L33
Non-nullable property 'LocalPath' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Code Quality Analysis
CodeQL Action v3 will be deprecated in December 2026. Please update all occurrences of the CodeQL Action in your workflow files to v4. For more information, see https://github.blog/changelog/2025-10-28-upcoming-deprecation-of-codeql-action-v3/
|