Skip to content

Update nuget non-major dependencies #273

Update nuget non-major dependencies

Update nuget non-major dependencies #273

Triggered via pull request September 26, 2024 02:11
Status Success
Total duration 3m 24s
Artifacts

build-and-analyze.yml

on: pull_request
Build, test & analyze
3m 16s
Build, test & analyze
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Clients/EmailServiceClient.cs#L33
Update this logger to use its enclosing type. (https://rules.sonarsource.com/csharp/RSPEC-6672)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Producers/CommonProducer.cs#L24
Update this logger to use its enclosing type. (https://rules.sonarsource.com/csharp/RSPEC-6672)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Consumers/KafkaConsumerBase.cs#L40
Log message template placeholder ' Class ' should only contain letters, numbers, and underscore. (https://rules.sonarsource.com/csharp/RSPEC-6674)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Consumers/KafkaConsumerBase.cs#L40
Log message template placeholder ' e.Reason ' should only contain letters, numbers, and underscore. (https://rules.sonarsource.com/csharp/RSPEC-6674)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Consumers/KafkaConsumerBase.cs#L62
Do not call 'GC.SuppressFinalize'. (https://rules.sonarsource.com/csharp/RSPEC-3971)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Producers/CommonProducer.cs#L57
Log message template placeholder 'result.Status' should only contain letters, numbers, and underscore. (https://rules.sonarsource.com/csharp/RSPEC-6674)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Producers/CommonProducer.cs#L101
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
Build, test & analyze: src/Altinn.Notifications.Email.Integrations/Clients/EmailServiceClient.cs#L100
Await WaitForCompletionResponseAsync instead. (https://rules.sonarsource.com/csharp/RSPEC-6966)
Build, test & analyze: src/Altinn.Notifications.Email/Controllers/DeliveryReportController.cs#L68
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
Build, test & analyze: src/Altinn.Notifications.Email/Program.cs#L117
Make sure publicly writable directories are used safely here. (https://rules.sonarsource.com/csharp/RSPEC-5443)