🐛 Bug - Fix codeQL analysis #17
Annotations
1 error and 11 warnings
Analyze (csharp)
Encountered a fatal error while running "/opt/hostedtoolcache/CodeQL/2.14.6/x64/codeql/codeql database finalize --finalize-dataset --threads=2 --ram=5897 /home/runner/work/_temp/codeql_databases/csharp". Exit code was 32 and error was: CodeQL detected code written in C# but could not process any of it. Review our troubleshooting guide at https://gh.io/troubleshooting-code-scanning/no-source-code-seen-during-build.
|
Analyze (csharp)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Summary.cs#L5
Non-nullable property 'Emails' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Summary.cs#L6
Non-nullable property 'Meetings' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Summary.cs#L7
Non-nullable property 'Summary' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Summary.cs#L8
Non-nullable property 'ModelUsed' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Meeting.cs#L6
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Classes/Meeting.cs#L8
Non-nullable property 'Sender' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Services/LangChainAiService.cs#L7
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
Analyze (csharp):
src/TimesheetGPT.Application/Services/GraphService.cs#L105
Non-nullable property 'Attendees' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
Analyze (csharp):
src/TimesheetGPT.Application/Services/GraphService.cs#L43
Nullability of reference types in value of type 'List<string?>' doesn't match target type 'List<string>'.
|
Analyze (csharp):
src/TimesheetGPT.Application/Services/GraphService.cs#L69
Possible null reference assignment.
|