This repository has been archived by the owner on Jul 9, 2024. It is now read-only.

Sonarcloud

Merge pull request #225 from microsoft/feature/code-of-conduct #304

Sign in to view logs

Triggered via push May 14, 2024 12:38

baywet

main

Status Success

Total duration 2m 10s

Artifacts –

sonarcloud.yml

on: push

check if SONAR_TOKEN is set in github secrets

Annotations

10 warnings

Build: src/JsonParseNode.cs#L295

All 'GetUntypedValue' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)

Build: src/JsonSerializationWriter.cs#L25

Fix this implementation of 'IDisposable' to conform to the dispose pattern. (https://rules.sonarsource.com/csharp/RSPEC-3881)

Build: src/JsonSerializationWriter.cs#L25

'ISerializationWriter' implements 'IDisposable' so 'IDisposable' can be removed from the inheritance list. (https://rules.sonarsource.com/csharp/RSPEC-1939)

Build: src/JsonSerializationWriter.cs#L369

Refactor this method to reduce its Cognitive Complexity from 24 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)

Build: src/JsonSerializationWriter.cs#L372

Use a comparison to 'default(T)' instead or add a constraint to 'T' so that it can't be a value type. (https://rules.sonarsource.com/csharp/RSPEC-2955)

Build: src/JsonSerializationWriter.cs#L378

Use a comparison to 'default(T)' instead or add a constraint to 'T' so that it can't be a value type. (https://rules.sonarsource.com/csharp/RSPEC-2955)

Build: src/JsonSerializationWriter.cs#L393

Use a comparison to 'default(T)' instead or add a constraint to 'T' so that it can't be a value type. (https://rules.sonarsource.com/csharp/RSPEC-2955)

Build: src/JsonSerializationWriter.cs#L408

Use a comparison to 'default(T)' instead or add a constraint to 'T' so that it can't be a value type. (https://rules.sonarsource.com/csharp/RSPEC-2955)

Build: src/JsonSerializationWriter.cs#L430

Use a comparison to 'default(T)' instead or add a constraint to 'T' so that it can't be a value type. (https://rules.sonarsource.com/csharp/RSPEC-2955)

Build: src/JsonSerializationWriter.cs#L508

Remove this unnecessary check for null. (https://rules.sonarsource.com/csharp/RSPEC-2589)