Bart Koelman

Results 95 issues of Bart Koelman

Address existing violations of [S2931: Classes with "IDisposable" members should implement "IDisposable"](https://rules.sonarsource.com/csharp/RSPEC-2931) 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
Type/code-quality

Address existing suppressions of [S2699: Tests should include assertions](https://rules.sonarsource.com/csharp/RSPEC-2699) in the codebase. To find existing violations, search for "S2699" in `src/Steeltoe.All.sln` to find existing suppressions. To address the violations, choose...

Type/code-quality

Address existing violations of [S2360: Optional parameters should not be used](https://rules.sonarsource.com/csharp/RSPEC-2360) 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 [S2292: Trivial properties should be auto-implemented](https://rules.sonarsource.com/csharp/RSPEC-2292) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing violations, enable the rule (see...

Warn/breaking-change
Type/code-quality

Address existing violations of [S2221: "Exception" should not be caught when not required by called methods](https://rules.sonarsource.com/csharp/RSPEC-2221) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find...

Type/code-quality

Address existing violations of [S1694: An abstract class should have both abstract and concrete methods](https://rules.sonarsource.com/csharp/RSPEC-1694) in the codebase and set severity to `Warning` in `Steeltoe.Debug.ruleset` and `Steeltoe.Release.ruleset`. To find existing...

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

Address existing violations of [S138: Functions should not have too many lines of code](https://rules.sonarsource.com/csharp/RSPEC-138) 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 [S134: Control flow statements "if", "switch", "for", "foreach", "while", "do" and "try" should not be nested too deeply](https://rules.sonarsource.com/csharp/RSPEC-134) in the codebase and set severity to `Warning`...

Type/code-quality

After all Sonar rule issues have been addressed, activate [S1309: Track uses of in-source issue suppressions](https://rules.sonarsource.com/csharp/RSPEC-***) temporarily by setting severity to `Warning` in `Steeltoe.Debug.ruleset` and rebuild `src/Steeltoe.All.sln` to make them...

Type/code-quality

Address existing violations of [S125: Sections of code should not be commented out](https://rules.sonarsource.com/csharp/RSPEC-125) in the codebase and set severity to `Info` in `Steeltoe.Debug.ruleset` and to `Warning` in `Steeltoe.Release.ruleset`. Note: The...

Type/code-quality