Skip to content

Removed some code repetition, simplified forced pause/slowdown check #155

Removed some code repetition, simplified forced pause/slowdown check

Removed some code repetition, simplified forced pause/slowdown check #155

Triggered via pull request August 26, 2024 18:48
Status Success
Total duration 50s
Artifacts

pr-check.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

11 warnings
Builds
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3.0.3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Builds: Source/Common/Syncing/Logger/LoggingByteWriter.cs#L51
Nullability of type of parameter 'bytes' doesn't match overridden member (possibly because of nullability attributes).
Builds: Source/Common/Syncing/Logger/LoggingByteWriter.cs#L58
Nullability of type of parameter 's' doesn't match overridden member (possibly because of nullability attributes).
Builds: Source/Common/Native.cs#L36
Cannot convert null literal to non-nullable reference type.
Builds: Source/Common/Native.cs#L36
Cannot convert null literal to non-nullable reference type.
Builds: Source/Common/Native.cs#L39
Cannot convert null literal to non-nullable reference type.
Builds: Source/Common/Native.cs#L39
Cannot convert null literal to non-nullable reference type.
Builds: Source/Common/Util/ServerLog.cs#L8
Non-nullable field 'info' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
Builds: Source/Common/Util/ServerLog.cs#L9
Non-nullable field 'error' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the field as nullable.
Builds: Source/Common/ChatCommands.cs#L7
Possible null reference return.
Builds: Source/Common/Syncing/Worker/ReadingSyncWorker.cs#L17
Converting null literal or possible null value to non-nullable type.