Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mirror: Code Cleanup: Purge Obsolete MapManager Methods #300

Conversation

SimpleStation14
Copy link
Member

Mirror of PR #26279: Code Cleanup: Purge obsolete MapManager methods from space-wizards space-wizards/space-station-14

ae3d745430e3c5250d19553429e7cda2b8050d10

PR opened by Tayrtahn at 2024-03-20 02:51:49 UTC


PR changed 49 files with 97 additions and 122 deletions.

The PR had the following labels:

  • Status: Needs Review

Original Body

About the PR

Cleans up some obsolete method calls.

Why / Balance

This is definitely low -hanging fruit on the code cleanup tree, but clearing out compiler warnings is always nice.

Technical details

Removes every use of obsolete MapManager methods (GetGrid, GridExists, and TryGetGrid), replacing them with appropriate EntityManager methods or their corresponding EntitySystem extension methods.

Media

Code

  • I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

@SimpleStation14 SimpleStation14 added the Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot label Apr 22, 2024
Copy link
Contributor

@DangerRevolution DangerRevolution left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no idea what this does

@SimpleStation14 SimpleStation14 marked this pull request as draft May 4, 2024 21:12
@VMSolidus VMSolidus marked this pull request as ready for review May 12, 2024 03:15
@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label May 20, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot added Changes: C# Changes any cs files and removed Status: Merge Conflict FIX YOUR PR AAAGH labels May 29, 2024
@SimpleStation14 SimpleStation14 changed the title Mirror: Code Cleanup: Purge obsolete MapManager methods Mirror: Code Cleanup: Purge Obsolete MapManager Methods Jul 9, 2024
@VMSolidus
Copy link
Member

I'm going to cherrypick my way through the failed mirrors and see if that fixes this first.

@github-actions github-actions bot added the Status: Merge Conflict FIX YOUR PR AAAGH label Jul 12, 2024
Copy link
Contributor

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@github-actions github-actions bot removed the Status: Merge Conflict FIX YOUR PR AAAGH label Jul 12, 2024
@DEATHB4DEFEAT
Copy link
Member

2 files

@VMSolidus
Copy link
Member

This was included in the engine PRs, and as such is no longer needed.

@VMSolidus VMSolidus closed this Jul 12, 2024
Mnemotechnician pushed a commit to Mnemotechnician/Einstein-Engines that referenced this pull request Dec 15, 2024
Mnemotechnician pushed a commit to Mnemotechnician/Einstein-Engines that referenced this pull request Dec 15, 2024
Mnemotechnician pushed a commit to Mnemotechnician/Einstein-Engines that referenced this pull request Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Pull Request Mirror Mirrors a PR from another Repo. Automatically applied by mirror bot
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants