-
Notifications
You must be signed in to change notification settings - Fork 45
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
Add miscellaneous mappings #275
Conversation
Should the class_1015 and class_0_681 in net.minecraft.client instead? |
Yeah, forgot to update packages when I originally mapped them |
Can you update your branch with latest changes I made? It fixes workflow. I'll review your pr when it's done. |
Updated, looks like it needs maintainer approval before the workflow is run. |
Is this spell check fail normal? looks like most of the files flagged aren't relevant to this PR. |
Yes, ignore spellcheck for v2 branch, the config needs to be updated for it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, I'm gonna wait for the workflow to be done and merge if it's successful (ignoring the 1.14.x one).
mappings/net/minecraft/predicate/block/BlockStatePredicate.mapping
Outdated
Show resolved
Hide resolved
Co-authored-by: Cat Core <[email protected]>
This PR consists of mappings based on 1.20.4 fabric (yarn mappings) for 1.8.9.