Add a analyzer for Harmony to help detect issues with patches - #910
Draft
mibac138 wants to merge 1 commit into
Draft
Add a analyzer for Harmony to help detect issues with patches#910mibac138 wants to merge 1 commit into
mibac138 wants to merge 1 commit into
Annotations
31 warnings
|
Node.js 20 actions are deprecated. The following actions are running on Node.js 20 and may not work as expected: actions/checkout@v4, actions/setup-dotnet@v4. Actions will be forced to run with Node.js 24 by default starting June 2nd, 2026. Node.js 20 will be removed from the runner on September 16th, 2026. Please check if updated versions of these actions are available that support Node.js 24. To opt into Node.js 24 now, set the FORCE_JAVASCRIPT_ACTIONS_TO_NODE24=true environment variable on the runner or in your workflow file. Once Node.js 24 becomes the default, you can temporarily opt out by setting ACTIONS_ALLOW_USE_UNSECURE_NODE_VERSION=true. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
Source/Common/ReplayInfo.cs#L11
Non-nullable field 'name' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
Source/Common/ServerPlayer.cs#L37
Possible null reference return.
|
|
Source/Common/ScribeLike.cs#L5
Non-nullable field 'provider' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
Source/Common/CommandHandler.cs#L89
The switch expression does not handle some values of its input type (it is not exhaustive) involving an unnamed enum value. For example, the pattern '(Multiplayer.Common.DevModeScope)2' is not covered.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L22
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L22
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Networking/MpConnectionState.cs#L14
Possible null reference return.
|
|
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L81
Possible null reference assignment.
|
|
Source/Common/Networking/MpConnectionState.cs#L14
Possible null reference return.
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L24
Non-nullable field 'addGenericMethod' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L22
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L22
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Util/TomlSettings.cs#L69
Possible null reference assignment.
|
|
Source/Common/Util/TomlSettings.cs#L67
Dereference of a possibly null reference.
|
|
Source/Common/CommandHandler.cs#L89
The switch expression does not handle some values of its input type (it is not exhaustive) involving an unnamed enum value. For example, the pattern '(Multiplayer.Common.DevModeScope)2' is not covered.
|
|
Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Syncing/Worker/SyncWorkerDictionaryTree.cs#L91
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/SyncWorkerDictionaryTree.cs#L84
Possible null reference assignment.
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L116
Dereference of a possibly null reference.
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L115
Converting null literal or possible null value to non-nullable type.
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L81
Possible null reference assignment.
|
|
Source/Common/Util/TomlSettings.cs#L69
Possible null reference assignment.
|
|
Source/Common/Util/TomlSettings.cs#L67
Dereference of a possibly null reference.
|
|
Source/Common/Syncing/Worker/SyncWorkerEntry.cs#L24
Non-nullable field 'addGenericMethod' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
|
|
Source/Common/CommandHandler.cs#L89
The switch expression does not handle some values of its input type (it is not exhaustive) involving an unnamed enum value. For example, the pattern '(Multiplayer.Common.DevModeScope)2' is not covered.
|
The logs for this run have expired and are no longer available.
Loading