Skip to content

Improved source generators #489

Improved source generators

Improved source generators #489

Triggered via pull request October 14, 2023 14:40
Status Success
Total duration 6m 6s
Artifacts

sonar-code-analysis.yml

on: pull_request
Build and analyze
5m 51s
Build and analyze
Fit to window
Zoom out
Zoom in

Annotations

10 warnings
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/SourceBuilder.cs#L148
Change return type to 'void'; not a single caller uses the returned value. (https://rules.sonarsource.com/csharp/RSPEC-3241)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/HashCode.cs#L72
Remove this 'throw' expression. (https://rules.sonarsource.com/csharp/RSPEC-3877)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/HashCode.cs#L70
Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/HashCode.cs#L77
Remove this 'throw' expression. (https://rules.sonarsource.com/csharp/RSPEC-3877)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/HashCode.cs#L75
Do not forget to remove this deprecated code someday. (https://rules.sonarsource.com/csharp/RSPEC-1133)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/HashCode.cs#L43
Rename 'length' which hides the field with the same name. (https://rules.sonarsource.com/csharp/RSPEC-1117)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/EquatableArray.cs#L77
Rename 'array' which hides the field with the same name. (https://rules.sonarsource.com/csharp/RSPEC-1117)
Build and analyze: WoWsShipBuilder.Data.Generator/Utilities/EquatableArray.cs#L83
Rename 'array' which hides the field with the same name. (https://rules.sonarsource.com/csharp/RSPEC-1117)
Build and analyze: WoWsShipBuilder.Data.Generator.Test/DataElementAnalyzerTests/DataElementAnalyzerTest.cs#L18
Add at least one assertion to this test case. (https://rules.sonarsource.com/csharp/RSPEC-2699)
Build and analyze: WoWsShipBuilder.Data.Generator.Test/PropertyChangedGeneratorTests/PropertyChangedGeneratorTest.cs#L28
Add at least one assertion to this test case. (https://rules.sonarsource.com/csharp/RSPEC-2699)