Merge pull request #117 from Corvax-Frontier/up091124 #285
Zekins3366Annotations
32 warnings
|
Test Packaging
The following actions uses node12 which is deprecated and will be forced to run on node16: space-wizards/[email protected]. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Test Packaging
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/[email protected], space-wizards/[email protected], actions/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/MarkupNode.cs#L10
'MarkupNode' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ViewVariables/ViewVariablesTypeHandler.cs#L213
'Logger.ErrorS(string, Exception, string, params object?[])' is obsolete: 'Use ISawmill.Error'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/ZStd.cs#L288
The '&' operator should not be used on parameters or local variables in async methods.
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/ZStd.cs#L288
The '&' operator should not be used on parameters or local variables in async methods.
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Toolshed/TypeParsers/SessionTypeParser.cs#L46
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 Packaging:
RobustToolbox/Robust.Shared/Toolshed/TypeParsers/EnumTypeParser.cs#L49
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 Packaging:
RobustToolbox/Robust.Shared/Toolshed/TypeParsers/EntityTypeParser.cs#L38
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 Packaging:
RobustToolbox/Robust.Shared/Toolshed/TypeParsers/BoolTypeParser.cs#L52
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 Packaging:
RobustToolbox/Robust.Shared/Serialization/TypeSerializers/Implementations/Generic/ListSerializers.cs#L131
'Logger.Warning(string)' is obsolete: 'Use ISawmill.Warning'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Serialization/TypeSerializers/Implementations/Generic/ListSerializers.cs#L150
'Logger.Warning(string)' is obsolete: 'Use ISawmill.Warning'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/MarkupNode.cs#L10
'MarkupNode' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ComponentTrees/ComponentTreeSystem.cs#L289
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ViewVariables/ViewVariablesTypeHandler.cs#L213
'Logger.ErrorS(string, Exception, string, params object?[])' is obsolete: 'Use ISawmill.Error'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Containers/SharedContainerSystem.cs#L564
'TransformComponent.AttachToGridOrMap()' is obsolete: 'Use the system's method instead.'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Containers/ContainerManagerComponent.cs#L34
'Component.Owner' is obsolete: 'Update your API to allow accessing Owner through other means'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L254
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L261
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/CPUJob/JobQueues/Job.cs#L187
'Logger.ErrorS(string, string, params object?[])' is obsolete: 'Use ISawmill.Error'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ProfileOptSetup.cs#L16
Unreachable code detected
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/ZStd.cs#L288
The '&' operator should not be used on parameters or local variables in async methods.
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Utility/MarkupNode.cs#L10
'MarkupNode' overrides Object.Equals(object o) but does not override Object.GetHashCode()
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L254
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L261
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Containers/ContainerManagerComponent.cs#L34
'Component.Owner' is obsolete: 'Update your API to allow accessing Owner through other means'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ComponentTrees/ComponentTreeSystem.cs#L289
'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/CPUJob/JobQueues/Job.cs#L187
'Logger.ErrorS(string, string, params object?[])' is obsolete: 'Use ISawmill.Error'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/ViewVariables/ViewVariablesTypeHandler.cs#L213
'Logger.ErrorS(string, Exception, string, params object?[])' is obsolete: 'Use ISawmill.Error'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Containers/ContainerSlot.cs#L84
'Logger.Warning(string)' is obsolete: 'Use ISawmill.Warning'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Containers/SharedContainerSystem.cs#L564
'TransformComponent.AttachToGridOrMap()' is obsolete: 'Use the system's method instead.'
|
|
Test Packaging:
RobustToolbox/Robust.Shared/Console/Commands/MapCommands.cs#L120
'IMapManager.IsMapInitialized(MapId)' is obsolete: 'Use MapSystem'
|