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

Temporarily revert "Overmap shuttles no longer need empty sectors" #18978

Closed
wants to merge 1 commit into from

Conversation

NonQueueingMatt
Copy link
Contributor

Reverts #18898 temporarily while a fix is investigated as shuttles can't launch anymore

@NonQueueingMatt NonQueueingMatt added 🚫 Do Not Merge Test Merge Candidate The PR is eligible for test merging. labels Apr 17, 2024
@github-actions github-actions bot added the 🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. label Apr 17, 2024
Copy link

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ EDITORCONFIG editorconfig-checker 5 0 0.02s

See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

github-merge-queue bot pushed a commit that referenced this pull request Apr 18, 2024
Open space landmarks were placed on the wrong z-levels for shuttles not
using mapped zs. Fixes the issue of #18978
Copy link

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

@NonQueueingMatt NonQueueingMatt deleted the revert-18898-emptysectorx24 branch April 20, 2024 18:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changelog Required 🚫 Do Not Merge 🗺️ Mapping - Away Ship/Away Site The PR touches away ship and/or away site map files. Merge Conflict Test Merge Candidate The PR is eligible for test merging.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants