Skip to content

Code review fixes (#51) #10

Code review fixes (#51)

Code review fixes (#51) #10

Triggered via push October 7, 2023 01:50
Status Success
Total duration 2m 10s
Artifacts 1
Fit to window
Zoom out
Zoom in

Annotations

12 warnings
build
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1, actions/upload-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build: src/TimesheetGPT.Application/Models/Meeting.cs#L6
Non-nullable property 'Name' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: src/TimesheetGPT.Application/Models/Meeting.cs#L8
Non-nullable property 'Sender' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: src/TimesheetGPT.Application/Models/Summary.cs#L5
Non-nullable property 'Emails' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: src/TimesheetGPT.Application/Models/Summary.cs#L6
Non-nullable property 'Meetings' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: src/TimesheetGPT.Application/Models/Summary.cs#L7
Non-nullable property 'Summary' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: src/TimesheetGPT.Application/Models/Summary.cs#L8
Non-nullable property 'ModelUsed' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
build: 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.
build: 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.
build: src/TimesheetGPT.Application/Services/GraphService.cs#L43
Nullability of reference types in value of type 'List<string?>' doesn't match target type 'List<string>'.
build: src/TimesheetGPT.Application/Services/GraphService.cs#L69
Possible null reference assignment.
deploy
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/download-artifact@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/

Artifacts

Produced during runtime
Name Size
.net-app Expired
54 MB