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

Remove all unused shuttles files #2211

Merged
merged 2 commits into from
Oct 13, 2024

Conversation

dvir001
Copy link
Contributor

@dvir001 dvir001 commented Oct 13, 2024

About the PR

Cleaning inactive ship files

Copy link
Contributor

github-actions bot commented Oct 13, 2024

RSI Diff Bot; head commit c70dcbd merging into 583eca4
This PR makes changes to 1 or more RSIs. Here is a summary of all changes:

Resources/Textures/_NF/Objects/Misc/goldenrose.rsi

State Old New Status
icon Removed

Edit: diff updated after c70dcbd

Copy link
Contributor

@whatston3 whatston3 left a comment

Choose a reason for hiding this comment

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

Looking through the list, this seems alright. Would suggest keeping a commit ID to reference older ships if anyone wants to rescue them.

I'll get a Google sheet up for the removed ships for now, along with the last commit they were in and a link to the file on GItHub for that commit: https://docs.google.com/spreadsheets/d/16fskfbwOuW0kGL4b_lxUw8rjwe3y04XPECm3LYjV3bU/edit?gid=0#gid=0 If you want to put this on the wiki, go nuts, but I'm not doing that.

@dvir001 dvir001 merged commit 975002d into new-frontiers-14:master Oct 13, 2024
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Map-Shuttle Map - Shuttle No C# Sprites Status: Needs Review This PR is awaiting reviews
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants