Skip to content

Commit

Permalink
Just use .editorconfig, fewer files and it is more powerful
Browse files Browse the repository at this point in the history
  • Loading branch information
scottbilas committed Nov 21, 2023
1 parent 040017f commit 8818e7d
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 18 deletions.
21 changes: 21 additions & 0 deletions .editorconfig
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,27 @@ indent_size = 2
[*.{csproj,targets,props,config,runsettings,json,yml,yaml}]
indent_size = 2

[*.cs]

; public instance fields are ok when used with discretion
dotnet_diagnostic.CA1050.severity = silent
; top level names are fine
dotnet_diagnostic.CA1051.severity = silent
; don't care about this rule
dotnet_diagnostic.CA1720.severity = silent
; i accept preview features
dotnet_diagnostic.CA2252.severity = silent
; sealing types is not something i worry about (are there still perf implications for this in .net 7+?)
dotnet_diagnostic.CA1852.severity = silent

; locale warnings - i don't want to deal with these right now
dotnet_diagnostic.CA1304.severity = silent
dotnet_diagnostic.CA1305.severity = silent
dotnet_diagnostic.CA1310.severity = silent

;dotnet_diagnostic.CA1014.severity = silent
;dotnet_code_quality.CA1062.exclude_extension_method_this_parameter = true

[NiceIO.cs]
# TODO: go back and fix niceio
dotnet_diagnostic.CA1510.severity = silent
Expand Down
18 changes: 0 additions & 18 deletions .globalconfig

This file was deleted.

0 comments on commit 8818e7d

Please sign in to comment.