Bart Koelman

Results 95 issues of Bart Koelman

Address existing violations of [S1104: Fields should not have public accessibility](https://rules.sonarsource.com/csharp/RSPEC-1104) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing violations, enable the rule...

Warn/breaking-change
Warn/api-change
Type/code-quality

Address existing violations of [S104: Files should not have too many lines of code](https://rules.sonarsource.com/csharp/RSPEC-104) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing violations,...

Type/code-quality

Address existing violations of [S1135: Track uses of "TODO" tags](https://rules.sonarsource.com/csharp/RSPEC-1135) in the codebase and set severity to `Info` in `Steeltoe.Debug.ruleset` and to `Warning` in `Steeltoe.Release.ruleset`. Note: The difference in severities...

Type/code-quality

Address existing violations of [S1172: Unused method parameters should be removed](https://rules.sonarsource.com/csharp/RSPEC-1172) in the codebase and set severity to `Info` in `Steeltoe.Debug.ruleset` and to `Warning` in `Steeltoe.Release.ruleset`. Note: The difference in...

Type/code-quality

Address existing violations of [S106: Standard outputs should not be used directly to log anything](https://rules.sonarsource.com/csharp/RSPEC-106) in the codebase and set severity to `Info` in `Steeltoe.Debug.ruleset` and to `Warning` in `Steeltoe.Release.ruleset`....

Warn/breaking-change
Type/code-quality

Address existing violations of [S1200: Classes should not be coupled to too many other classes (Single Responsibility Principle)](https://rules.sonarsource.com/csharp/RSPEC-1200) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`....

Warn/breaking-change
Type/code-quality

Address existing violations of [Address S1168: Empty arrays and collections should be returned instead of null](https://rules.sonarsource.com/csharp/RSPEC-1168) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find...

Warn/breaking-change
Type/code-quality

Address existing violations of [S107: Methods should not have too many parameters](https://rules.sonarsource.com/csharp/RSPEC-107) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing violations, enable the...

Warn/breaking-change
Warn/api-change
Type/code-quality

Address existing violations of [S1067: Expressions should not be too complex](https://rules.sonarsource.com/csharp/RSPEC-1067) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing violations, enable the rule...

Type/code-quality

### Description When using [global usings](https://blog.jetbrains.com/dotnet/2021/11/18/global-usings-in-csharp-10/), analyzers no longer report any diagnostics. ### Expected behavior: Be able to use global usings, either: - From a source file `GlobalUsings.cs` that contains:...

enhancement