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

Add RegistryEntryAddedCallback.allEntries #4235

Merged

Conversation

modmuss50
Copy link
Member

RegistryEntryAddedCallback.allEntries invokes a callback for all previous and future entries in a registry. This is a common bit of boiler plate that many mods use.

@modmuss50 modmuss50 added registry-sync Pull requests and issues related to registry sync small change labels Nov 18, 2024
@modmuss50 modmuss50 added the last call If you care, make yourself heard right away! label Nov 18, 2024
@modmuss50 modmuss50 added the merge me please Pull requests that are ready to merge label Nov 25, 2024
@modmuss50 modmuss50 merged commit aa5b2ca into FabricMC:1.21.4 Nov 25, 2024
4 checks passed
modmuss50 added a commit that referenced this pull request Nov 25, 2024
* Add RegistryEntryAddedCallback.allEntries

* Pass a RegistryEntry.Reference

* Remove some temp test code

* Add note about recursion.

(cherry picked from commit aa5b2ca)
modmuss50 added a commit that referenced this pull request Nov 25, 2024
* Add RegistryEntryAddedCallback.allEntries

* Pass a RegistryEntry.Reference

* Remove some temp test code

* Add note about recursion.

(cherry picked from commit aa5b2ca)
(cherry picked from commit 2758bfb)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
last call If you care, make yourself heard right away! merge me please Pull requests that are ready to merge registry-sync Pull requests and issues related to registry sync small change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants