UI rework - very WIP, don't merge yet! #2545
Draft
Annotations
2 errors and 12 warnings
|
Test:
Wabbajack.DTOs.Test/ModListTests.cs#L86
There is no argument given that corresponds to the required parameter 'machineURL' of 'Client.GetDetailedStatus(string, string)'
|
|
Test
Process completed with exit code 1.
|
|
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/
|
|
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v2, actions/setup-dotnet@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Test:
Wabbajack.Paths.IO/KnownFolders.cs#L16
Unreachable code detected
|
|
Test:
Wabbajack.Downloaders.VerificationCache/VerificationCache.cs#L47
Nullability of reference types in return type of 'Task<(bool?, IDownloadState?)> VerificationCache.Get(IDownloadState archive)' doesn't match implicitly implemented member 'Task<(bool? IsValid, IDownloadState State)> IVerificationCache.Get(IDownloadState archive)'.
|
|
Test:
Wabbajack.Downloaders.VerificationCache/NullCache.cs#L7
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
|
Test:
Wabbajack.Networking.WabbajackClientApi/Client.cs#L274
Possible null reference return.
|
|
Test:
Wabbajack.Compression.BSA/DirectXTexUtil.cs#L522
A possible null value may not be used for a type marked with
|
|
Test:
Wabbajack.Compression.BSA/BA2Archive/DX10Entry.cs#L34
The field 'DX10Entry._unk16' is never used
|
|
Test:
Wabbajack.Downloaders.Mega/MegaToken.cs#L8
Non-nullable property 'Email' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test:
Wabbajack.Downloaders.Mega/MegaToken.cs#L11
Non-nullable property 'Password' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
|
|
Test:
Wabbajack.Hashing.PHash/TexConvImageLoader.cs#L158
The variable 'ex' is declared but never used
|
|
Test:
Wabbajack.Networking.Steam/Client.cs#L122
'SteamUser.UpdateMachineAuthCallback' is obsolete: 'Steam no longer sends machine auth as of 2023, use SteamAuthentication.'
|
Loading