Update the target .NET Framework to version 9.0 #338
Ahmed-Ghanambuild-and-analyze.yml
on: pull_request
Build, test & analyze
3m 19s
Annotations
2 errors and 11 warnings
|
Build, test & analyze
At least one failing test was found
|
|
Build, test & analyze
Process completed with exit code 1.
|
|
Build, test & analyze
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
|
|
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#L62
Do not call 'GC.SuppressFinalize'. (https://rules.sonarsource.com/csharp/RSPEC-3971)
|
|
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/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)
|