Skip to content

Merge pull request #189 from microsoft/bugfix/code-ql-permissions #296

Merge pull request #189 from microsoft/bugfix/code-ql-permissions

Merge pull request #189 from microsoft/bugfix/code-ql-permissions #296

Triggered via push February 5, 2024 16:46
Status Success
Total duration 2m 5s
Artifacts

sonarcloud.yml

on: push
check if SONAR_TOKEN is set in github secrets
0s
check if SONAR_TOKEN is set in github secrets
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Build: src/DefaultQueryParameters.cs#L10
Remove this empty class, write its code or make it an "interface". (https://rules.sonarsource.com/csharp/RSPEC-2094)
Build: src/NativeResponseHandler.cs#L20
Make this field 'private' and encapsulate it in a 'public' property. (https://rules.sonarsource.com/csharp/RSPEC-1104)
Build: src/NativeResponseWrapper.cs#L14
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
Build: src/RequestHeaders.cs#L60
Refactor 'Values' into a method, properties should not copy collections. (https://rules.sonarsource.com/csharp/RSPEC-2365)
Build: src/serialization/KiotaJsonSerializer.Serialization.cs#L21
All 'SerializeAsStream' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)
Build: src/serialization/KiotaJsonSerializer.Serialization.cs#L30
All 'SerializeAsString' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)
Build: src/store/BackingStoreFactorySingleton.cs#L10
Add a 'protected' constructor or the 'static' keyword to the class declaration. (https://rules.sonarsource.com/csharp/RSPEC-1118)
Build: src/serialization/ParsableFactory.cs#L15
Add the 'out' keyword to parameter 'T' to make it 'covariant'. (https://rules.sonarsource.com/csharp/RSPEC-3246)
Build: src/serialization/ParseNodeFactoryRegistry.cs#L36
Pass a timeout to limit the execution time. (https://rules.sonarsource.com/csharp/RSPEC-6444)
Build: src/serialization/KiotaSerializer.Deserialization.cs#L24
All 'Deserialize' method overloads should be adjacent. (https://rules.sonarsource.com/csharp/RSPEC-4136)