-
Notifications
You must be signed in to change notification settings - Fork 26
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
Alias wrench legacy nodes #246
Conversation
3224b56
to
c06dae6
Compare
c06dae6
to
f0a3f87
Compare
Click for detailed source code test coverage reportTest coverage report for Technic CNC 79.01% in 10/14 files:
Test coverage report for technic chests 45.24% in 6/6 files:
Test coverage report for technic 65.26% in 111/111 files:
Raw test runner output for geeks:CNC:
Chests:
Technic:
|
Managed to push c06dae6 without changing anything... |
Actually not sure about this, it seems that some nodes were not actually possible to pick up even while those are registered. wooden sign from default is probably only thing that might still be around, maybe. It might be better to just forget about these, all technic nodes are from older legacy.lua and these aliases were added around 8 years ago. |
Works for me 👍 |
Alias default:sign_wall was also added 6 years ago and probably not needed for wrench anymore. Closing this PR as useless, adding aliases that most likely are not needed anywhere. |
Fixes #245
Additionally I'd like to also add note here, imo in future wrench should not be registering any new nodes but just single item.