Skip to content

Commit 4f782e2

Browse files
committed
Set severity for IDE0005 to warning and add doc file
1 parent a96d9af commit 4f782e2

File tree

2 files changed

+22
-0
lines changed

2 files changed

+22
-0
lines changed

distribution/dotnet7/.editorconfig

+1
Original file line numberDiff line numberDiff line change
@@ -463,6 +463,7 @@ dotnet_diagnostic.CA1014.severity = none # https://github.com/atc-net
463463
dotnet_diagnostic.CA1068.severity = error # https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/MicrosoftCodeAnalysis/CA1068.md
464464
dotnet_diagnostic.CA1707.severity = error # https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/MicrosoftCodeAnalysis/CA1707.md
465465
dotnet_diagnostic.CA2007.severity = suggestion # https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/MicrosoftCodeAnalysis/CA2007.md
466+
dotnet_diagnostic.IDE0005.severity = warning # https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/MicrosoftCodeAnalysis/IDE0005.md
466467
dotnet_diagnostic.IDE0058.severity = none # https://github.com/atc-net/atc-coding-rules/blob/main/documentation/CodeAnalyzersRules/MicrosoftCodeAnalysis/IDE0058.md
467468

468469

Original file line numberDiff line numberDiff line change
@@ -0,0 +1,21 @@
1+
# Rule adding suggestion - Decision
2+
3+
### Rule summary
4+
5+
| | Value |
6+
| ----------- |------------------------------------------------|
7+
| Title | IDE0005 Remove unnecessary using directives |
8+
| CheckId | IDE0005 |
9+
| Category | Style |
10+
| Link | https://learn.microsoft.com/en-us/dotnet/fundamentals/code-analysis/style-rules/ide0005 |
11+
12+
### What is the problem
13+
This rule detects unused usings which can simply be removed. This rule is currently not a part of our coding rules and it should be.
14+
15+
### Decision
16+
17+
We update the `root/.editorconfig` with `severity = warning` and a doc-link.
18+
19+
### Reason & arguments
20+
21+
* This rule detects code that can be removed and no code is better than no code - see dissuction in [Issue #74](https://github.com/atc-net/atc-coding-rules/issues/74).

0 commit comments

Comments
 (0)