This repository was archived by the owner on Jul 28, 2025. It is now read-only.
fix: Resolve SonarQube warnings (#36) #373
github-merge-queue[bot]cicd-1-pull-request.yaml
on: push
Set CI/CD metadata
4s
Validate PR title
0s
Commit stage
/
Scan secrets
6s
Commit stage
/
Check file format
5s
Commit stage
/
Check Markdown format
6s
Commit stage
/
Check English usage
7s
Commit stage
/
Lint Terraform
8s
Commit stage
/
Count lines of code
7s
Matrix: Image build stage / build-and-push
Waiting for pending jobs
Image build stage
/
Append short commit hash to images
Image build stage
/
aggregate-json
Acceptance stage
/
Accessibility test
Acceptance stage
/
Contract test
Acceptance stage
/
Integration test
Acceptance stage
/
Load test
Acceptance stage
/
Security test
Acceptance stage
/
UI performance test
Acceptance stage
/
UI test
Acceptance stage
/
Environment tear down
Annotations
20 warnings
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L25
Non-nullable property 'RecipientOrgCode' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L23
Non-nullable property 'RecipientOdsCode' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L21
Non-nullable property 'RecipientName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L19
Non-nullable property 'Recipient' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L17
Non-nullable property 'UploadTimestamp' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L15
Non-nullable property 'ExpiryTime' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L13
Non-nullable property 'FileName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/OutboxService/TrackOutboxResponse.cs#L7
Non-nullable property 'MessageId' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MailboxConfigurationResolver.cs#L14
Converting null literal or possible null value to non-nullable type.
|
|
Test stage / Unit tests:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MeshConnectConfiguration.cs#L51
Non-nullable property 'ProxyAddress' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/MessageMetaData.cs#L6
Non-nullable property 'ToMailbox' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/MessageMetaData.cs#L5
Non-nullable property 'WorkflowID' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/Response/APIErrorResponse.cs#L11
Non-nullable property 'ErrorDescription' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/FileAttachment.cs#L7
Non-nullable property 'ContentType' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/FileAttachment.cs#L6
Non-nullable property 'Content' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/FileAttachment.cs#L5
Non-nullable property 'FileName' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Models/MeshResponse.cs#L8
Non-nullable field 'Response' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
Test stage / Perform static analysis:
src/dotnet-mesh-client/application/DotNetMeshClient/NHS.Mesh.Client/Configuration/MeshConnectConfiguration.cs#L51
Non-nullable property 'ProxyAddress' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test stage / Perform static analysis:
src/ServiceLayer.Common/Data/Migrations/20250521094738_AddNbssAppointmentEventTable.cs#L41
Define a constant instead of using this literal 'varchar(30)' 5 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
|
Test stage / Perform static analysis:
src/ServiceLayer.Common/Data/Migrations/20250521094738_AddNbssAppointmentEventTable.cs#L23
Define a constant instead of using this literal 'char(1)' 8 times. (https://rules.sonarsource.com/csharp/RSPEC-1192)
|
Artifacts
Produced during runtime
| Name | Size | Digest | |
|---|---|---|---|
|
lines-of-code-report.json.zip
Expired
|
889 Bytes |
sha256:ae49e0dcb16a0e02a7582d00f9b2741023603aac5edc8ae2f3e36106df1b2687
|
|