Add tests for error handling in CryptoQuotesReader #31
build-and-analyze.yml
on: push
Build, Test and Analyze
56s
Annotations
10 warnings
|
Build, Test and Analyze:
Taxes/TickerProcessing.cs#L85
Make 'ProcessNoop' a static method. (https://rules.sonarsource.com/csharp/RSPEC-2325)
|
|
Build, Test and Analyze:
Taxes/CryptoEventsReader.cs#L166
Refactor this method to reduce its Cognitive Complexity from 42 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Build, Test and Analyze:
Taxes/Event.cs#L233
Remove this 'throw' expression. (https://rules.sonarsource.com/csharp/RSPEC-3877)
|
|
Build, Test and Analyze:
Taxes/TickerState.cs#L253
Remove this 'throw' expression. (https://rules.sonarsource.com/csharp/RSPEC-3877)
|
|
Build, Test and Analyze:
Taxes/CryptoEventsReader.cs#L79
Refactor this method to reduce its Cognitive Complexity from 25 to the 15 allowed. (https://rules.sonarsource.com/csharp/RSPEC-3776)
|
|
Build, Test and Analyze:
Taxes/FileUtils.cs#L9
Make sure this weak hash algorithm is not used in a sensitive context here. (https://rules.sonarsource.com/csharp/RSPEC-4790)
|
|
Build, Test and Analyze:
Taxes/CryptoEventsReader.cs#L65
Indexing at 0 should be used instead of the "Enumerable" extension method "First" (https://rules.sonarsource.com/csharp/RSPEC-6608)
|
|
Build, Test and Analyze:
Taxes/TickerProcessing.cs#L5
Remove the unused private field 'Separator'. (https://rules.sonarsource.com/csharp/RSPEC-1144)
|
|
Build, Test and Analyze:
Taxes/TickerProcessing.cs#L562
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|
|
Build, Test and Analyze:
Taxes/TickerProcessing.cs#L383
Complete the task associated to this 'TODO' comment. (https://rules.sonarsource.com/csharp/RSPEC-1135)
|