Addressed some points in the review. · PolterTzi/space-station-14-fork@a506959

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/[email protected], space-wizards/[email protected], actions/[email protected]. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Build: RobustToolbox/Robust.Shared/Utility/MarkupNode.cs#L10

'MarkupNode' overrides Object.Equals(object o) but does not override Object.GetHashCode()

Build: RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L254

'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'

Build: RobustToolbox/Robust.Shared/Console/Commands/TeleportCommands.cs#L261

'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'

Build: RobustToolbox/Robust.Shared/ViewVariables/ViewVariablesTypeHandler.cs#L213

'Logger.ErrorS(string, Exception, string, params object?[])' is obsolete: 'Use ISawmill.Error'

Build: RobustToolbox/Robust.Shared/ComponentTrees/ComponentTreeSystem.cs#L291

'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'

Build: RobustToolbox/Robust.Shared/CPUJob/JobQueues/Job.cs#L187

'Logger.ErrorS(string, string, params object?[])' is obsolete: 'Use ISawmill.Error'

Build: RobustToolbox/Robust.Shared/Containers/ContainerManagerComponent.cs#L34

'Component.Owner' is obsolete: 'Update your API to allow accessing Owner through other means'

Build: RobustToolbox/Robust.Shared/Console/Commands/MapCommands.cs#L120

'IMapManager.IsMapInitialized(MapId)' is obsolete: 'Use MapSystem'

Build: RobustToolbox/Robust.Shared/Console/Commands/MapCommands.cs#L91

'IMapManager.DeleteGrid(EntityUid)' is obsolete: 'Just delete the grid entity'

Build: RobustToolbox/Robust.Shared/Console/Commands/MapCommands.cs#L146

'IMapManager.GetMapEntityId(MapId)' is obsolete: 'Use TryGetMap'

Provide feedback