Bump a lot of deps to their 8.0 versions (#777) #168
3079 new problems found by Qodana for .NET
Qodana for .NET
3079 new problems were found
| Inspection name | Severity | Problems |
|---|---|---|
Unexpected text token error |
🔴 Failure | 84 |
Possible null reference assignment. |
🔴 Failure | 58 |
Dereference of a possibly null reference. |
🔴 Failure | 7 |
Possible null reference argument for a parameter. |
🔴 Failure | 4 |
The switch expression does not handle all possible inputs (it is not exhaustive). |
🔴 Failure | 1 |
Possible null reference return. |
🔴 Failure | 1 |
Redundant using directive |
🔶 Warning | 1341 |
Auto-property accessor is never used (non-private accessibility) |
🔶 Warning | 99 |
Invalid XML documentation comment |
🔶 Warning | 78 |
Usage of <inheritdoc /> is invalid |
🔶 Warning | 77 |
Redundant 'partial' modifier on type declaration |
🔶 Warning | 74 |
Cannot resolve reference in XML comment |
🔶 Warning | 48 |
Method return value is never used (private accessibility) |
🔶 Warning | 22 |
Namespace does not correspond to file location |
🔶 Warning | 18 |
Redundant namespace alias |
🔶 Warning | 12 |
Non-accessed field (private accessibility) |
🔶 Warning | 10 |
Static resource not resolved |
🔶 Warning | 9 |
Resource is not overridden in specific culture |
🔶 Warning | 8 |
Async function without await expression |
🔶 Warning | 7 |
Declaring new protected member in sealed class is the same as declaring it as private |
🔶 Warning | 6 |
XML comment has a 'param' tag for 'Parameter', but there is no parameter by that name |
🔶 Warning | 6 |
Redundant member initializer |
🔶 Warning | 6 |
Unused parameter (private accessibility) |
🔶 Warning | 6 |
Unused local variable |
🔶 Warning | 6 |
Parameter has no matching param tag in the XML comment |
🔶 Warning | 5 |
Function is recursive on all execution paths |
🔶 Warning | 5 |
Non-accessed positional property (non-private accessibility) |
🔶 Warning | 5 |
Private field can be converted into local variable |
🔶 Warning | 5 |
Type member is never used (private accessibility) |
🔶 Warning | 5 |
Add/remove 'this.' qualifier |
🔶 Warning | 3 |
Redundant explicit type in array creation |
🔶 Warning | 3 |
Auto-property accessor is never used (private accessibility) |
🔶 Warning | 3 |
Local function return value is never used |
🔶 Warning | 3 |
Code is unreachable |
🔶 Warning | 2 |
Operator '==' or operator '!=' with 'Object.Equals(object o)' and 'Object.GetHashCode()' not overridden |
🔶 Warning | 2 |
Non-accessed positional property (private accessibility) |
🔶 Warning | 2 |
Redundant name qualifier |
🔶 Warning | 2 |
Variable in local function hides variable from outer scope |
🔶 Warning | 2 |
Access to disposed captured variable |
🔶 Warning | 1 |
Expression is always 'true' or 'false' according to nullable reference types' annotations |
🔶 Warning | 1 |
Empty namespace declaration |
🔶 Warning | 1 |
Heuristically unreachable code |
🔶 Warning | 1 |
Possibly impure struct method is called on readonly variable: struct value always copied before invocation |
🔶 Warning | 1 |
Assignment is not used |
🔶 Warning | 1 |
Static field or auto-property in generic type |
🔶 Warning | 1 |
Static member initializer refers to static member below or in other part |
🔶 Warning | 1 |
Type member is never used (non-private accessibility) |
◽️ Notice | 386 |
Class is never instantiated (non-private accessibility) |
◽️ Notice | 144 |
Type member is never accessed via base type (non-private accessibility) |
◽️ Notice | 112 |
Member can be made private (non-private accessibility) |
◽️ Notice | 102 |
Type is never used (non-private accessibility) |
◽️ Notice | 57 |
Use preferred style for trailing comma before new line in multiline lists |
◽️ Notice | 52 |
Method return value is never used (non-private accessibility) |
◽️ Notice | 47 |
Redundant empty argument list on object creation expression |
◽️ Notice | 24 |
Unused parameter (non-private accessibility) |
◽️ Notice | 16 |
Invert 'if' statement to reduce nesting |
◽️ Notice | 13 |
Resource is overridden with identical value |
◽️ Notice | 10 |
Non-accessed field (non-private accessibility) |
◽️ Notice | 8 |
Member can be made protected (non-private accessibility) |
◽️ Notice | 5 |
Redundant 'else' keyword |
◽️ Notice | 5 |
Replace auto-property with computed property |
◽️ Notice | 5 |
Use explicit or implicit modifier definition for type members |
◽️ Notice | 4 |
Field can be made readonly (non-private accessibility) |
◽️ Notice | 4 |
Field can be made readonly (private accessibility) |
◽️ Notice | 4 |
Use explicit or implicit modifier definition for types |
◽️ Notice | 3 |
Put local function after 'return' or 'continue' |
◽️ Notice | 3 |
Parameter can be declared with the base type |
◽️ Notice | 3 |
Type parameter could be declared as covariant or contravariant |
◽️ Notice | 3 |
Use preferred style of 'new' expression when created type is evident |
◽️ Notice | 2 |
Remove redundant parentheses |
◽️ Notice | 2 |
Class with virtual (overridable) members never inherited (non-private accessibility) |
◽️ Notice | 2 |
Convert constructor into member initializers |
◽️ Notice | 2 |
'if-return' statement can be rewritten as 'return' statement |
◽️ Notice | 2 |
Some values of the enum are not processed inside 'switch' expression and are handled via exception in default arm |
◽️ Notice | 2 |
Use discard assignment |
◽️ Notice | 2 |
Use preferred namespace body style |
◽️ Notice | 1 |
Use preferred style of 'new' expression when created type is not evident |
◽️ Notice | 1 |
Convert property into auto-property (when possible) |
◽️ Notice | 1 |
Member can be made static (shared) (non-private accessibility) |
◽️ Notice | 1 |
Member can be made static (shared) (private accessibility) |
◽️ Notice | 1 |
Make constructor in abstract class protected |
◽️ Notice | 1 |
Redundant semicolon after type or namespace declaration |
◽️ Notice | 1 |
Redundant lambda signature parentheses |
◽️ Notice | 1 |
Some values of the enum are not processed inside 'switch' statement and are handled via default section |
◽️ Notice | 1 |
Convert 'as' expression type check and the following null check into negated pattern matching |
◽️ Notice | 1 |
Use raw string |
◽️ Notice | 1 |
Literal length can be reduced by using verbatim string |
◽️ Notice | 1 |
💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report
Contact Qodana team
Contact us at [email protected]
- Or via our issue tracker: https://jb.gg/qodana-issue
- Or share your feedback: https://jb.gg/qodana-discussions
Details
This result was published with Qodana GitHub Action
Annotations
Check warning on line 82 in src/Games/NexusMods.Games.BethesdaGameStudios/PluginAnalyzer.cs
github-actions / Qodana for .NET
Access to disposed captured variable
Captured variable is disposed in the outer scope
github-actions / Qodana for .NET
Use preferred namespace body style
Convert to file-scoped namespace
Check notice on line 74 in src/NexusMods.DataModel/Games/GamePath.cs
github-actions / Qodana for .NET
Use preferred style of 'new' expression when created type is evident
Redundant type specification
Check notice on line 25 in src/NexusMods.CLI/Verbs/DownloadLinks.cs
github-actions / Qodana for .NET
Use preferred style of 'new' expression when created type is evident
Redundant type specification
Check notice on line 18 in src/Games/NexusMods.Games.Sifu/Sifu.cs
github-actions / Qodana for .NET
Use preferred style of 'new' expression when created type is not evident
Missing type specification
Check notice on line 138 in src/NexusMods.DataModel/Games/AGame.cs
github-actions / Qodana for .NET
Remove redundant parentheses
Redundant parentheses
github-actions / Qodana for .NET
Remove redundant parentheses
Redundant parentheses
Check warning on line 61 in src/NexusMods.App.UI/Overlays/IOverlayController.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant
Check warning on line 66 in src/NexusMods.App.UI/Overlays/IOverlayController.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant
Check warning on line 67 in src/NexusMods.App.UI/Overlays/IOverlayController.cs
github-actions / Qodana for .NET
Add/remove 'this.' qualifier
Qualifier 'this.' is redundant
Check notice on line 193 in src/NexusMods.DataModel/ArchiveInstaller.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 20 in src/Networking/NexusMods.Networking.NexusWebApi.NMA/TypeFinder.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 35 in src/Networking/NexusMods.Networking.NexusWebApi/Verbs/NexusApiVerify.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 19 in src/NexusMods.App.UI/Controls/Spine/SpineDesignViewModel.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 66 in src/Games/NexusMods.Games.StardewValley/StardewValley.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 88 in src/NexusMods.StandardGameLocators/Services.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 14 in src/Games/NexusMods.Games.TestHarness/TypeFinder.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 31 in src/NexusMods.StandardGameLocators/EpicLocator.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 15 in src/Games/NexusMods.Games.BethesdaGameStudios/TypeFinder.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
Check notice on line 143 in src/Networking/NexusMods.Networking.NexusWebApi.NMA/OAuth.cs
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style
github-actions / Qodana for .NET
Use preferred style for trailing comma before new line in multiline lists
Remove trailing comma to conform to code style