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

Move all uncraft recipes (except unpacking) into uncraft folder #2349

Merged
merged 3 commits into from
Feb 28, 2023

Conversation

Lamandus
Copy link
Contributor

This is a double whammy. It moves all uncraft recipes into the uncraft folder (where it belongs) and also sane checks a lot of recipes (for example 5 hours for an AI array for a bot, giving not much stuff for the time wasted).

…ncraft recipes (except unpacking) into uncraft folder
@github-actions github-actions bot added the data PRs related to datas. Won't crash game (probably) label Feb 16, 2023
@Coolthulhu Coolthulhu self-assigned this Feb 19, 2023
@Lamandus
Copy link
Contributor Author

this will conflict with the #2345 , so either this should merge first, or the other, repectively

@Lamandus
Copy link
Contributor Author

conflicts resolved

@Coolthulhu
Copy link
Member

Having both changes happen in one commit makes it harder to review...

I'll need to figure out how to compare just the changed times.

@Lamandus
Copy link
Contributor Author

I will split the PR then, no problem

@Lamandus Lamandus changed the title Lowered "uncraft" recipes times to more sensible times AND move all uncraft recipes (except unpacking) into uncraft folder Move all uncraft recipes (except unpacking) into uncraft folder Feb 26, 2023
@Coolthulhu Coolthulhu merged commit 4b6fa84 into cataclysmbnteam:upload Feb 28, 2023
@Lamandus Lamandus deleted the robotdisassam branch March 2, 2023 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data PRs related to datas. Won't crash game (probably)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants