Some performance and code optimizations (#955) #298
Annotations
12 warnings
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v2, actions/setup-dotnet@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
dotnet build Release:
CompatBot/Commands/Sudo.cs#L137
'ILoggerBase.Factory' is obsolete: 'Factory-property is hard to mock for ILogger-interface. Instead use Logger.Factory. Marked obsolete with NLog v5.3'
|
|
dotnet build Release:
CompatBot/Utils/Extensions/AzureDevOpsClientExtensions.cs#L221
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/Utils/Extensions/AzureDevOpsClientExtensions.cs#L251
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/Utils/Extensions/AzureDevOpsClientExtensions.cs#L278
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/EventHandlers/LogParsing/ArchiveHandlers/ZipHandler.cs#L44
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/Commands/Sudo.Bot.cs#L251
'ILoggerBase.Factory' is obsolete: 'Factory-property is hard to mock for ILogger-interface. Instead use Logger.Factory. Marked obsolete with NLog v5.3'
|
|
dotnet build Release:
CompatBot/EventHandlers/LogParsing/ArchiveHandlers/SevenZipHandler.cs#L43
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/EventHandlers/LogParsing/ArchiveHandlers/RarHandler.cs#L43
Dereference of a possibly null reference.
|
|
dotnet build Release:
CompatBot/Utils/ColorGetter.cs#L58
This call site is reachable on: 'Windows' all versions. 'Bitmap' is only supported on: 'windows' 6.1 and later. (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca1416)
|
|
dotnet build Release:
CompatBot/Database/DbImporter.cs#L78
Method 'ExecuteSqlRawAsync' inserts interpolated strings directly into the SQL, without any protection against SQL injection. Consider using 'ExecuteSqlAsync' instead, which protects against SQL injection, or make sure that the value is sanitized and suppress the warning.
|
Loading