Skip to content

Add VisitableMappingTree interface #32

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.

Sign up for GitHub

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

Jump to bottom
Merged
modmuss50 merged 1 commit into from
Jun 30, 2023
Merged

Add VisitableMappingTree interface #32

modmuss50 merged 1 commit into from
Jun 30, 2023

Conversation

@NebelNidas
Copy link
Member

@NebelNidas NebelNidas commented May 13, 2023 •
edited
Loading

This allows greater flexibility, as consumers no longer have to rely on a specific MappingTree implementation directly (MemoryMappingTree).

@NebelNidas NebelNidas force-pushed the VisitableMappingTree branch from 9319db5 to 1fd5bbd Compare May 13, 2023 23:00
@modmuss50 modmuss50 requested a review from Player3324 May 16, 2023 10:43
@NebelNidas NebelNidas force-pushed the VisitableMappingTree branch from 1fd5bbd to 2a39d0a Compare June 16, 2023 08:51
@modmuss50 modmuss50 merged commit f92f89c into FabricMC:master Jun 30, 2023
@NebelNidas NebelNidas added this to the 0.5 milestone Oct 3, 2023
@NebelNidas NebelNidas deleted the VisitableMappingTree branch August 28, 2024 21:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@modmuss50 modmuss50 modmuss50 approved these changes

@Player3324 Player3324 Awaiting requested review from Player3324

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.5

Development

Successfully merging this pull request may close these issues.

2 participants

@NebelNidas @modmuss50