Skip to content

Implemented registry reader #6

Implemented registry reader

Implemented registry reader #6

Triggered via pull request July 5, 2024 12:17
Status Success
Total duration 2m 19s
Artifacts

sonarcloud.yaml

on: pull_request
analyse  /  sonar-analysis
2m 9s
analyse / sonar-analysis
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/UriOptionsLoader/UriOptionsLoaderService.cs#L35
Dispose '_changeTokenSource' when it is no longer needed. (https://rules.sonarsource.com/csharp/RSPEC-2930)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/RegistryService.cs#L15
Make '_registries' 'readonly'. (https://rules.sonarsource.com/csharp/RSPEC-2933)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/UriOptionsLoader/UriOptionsLoaderService.cs#L101
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/UriOptionsLoader/UriOptionsLoaderService.cs#L103
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/RegistryService.cs#L51
'System.Exception' should not be thrown by user code. (https://rules.sonarsource.com/csharp/RSPEC-112)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/BlockReader/BlockReaderJob.cs#L148
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/BlockReader/BlockReaderJob.cs#L176
Use PascalCase for named placeholders. (https://rules.sonarsource.com/csharp/RSPEC-6678)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/BlockReader/BlockReaderBackgroundService.cs#L44
Either log this exception and handle it, or rethrow it with some contextual information. (https://rules.sonarsource.com/csharp/RSPEC-2139)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Test/ChroniclerRepositoryTests.cs#L175
Add at least one assertion to this test case. (https://rules.sonarsource.com/csharp/RSPEC-2699)
analyse / sonar-analysis: src/ProjectOrigin.Chronicler.Server/UriOptionsLoader/UriOptionsLoaderService.cs#L35
Dispose '_changeTokenSource' when it is no longer needed. (https://rules.sonarsource.com/csharp/RSPEC-2930)