Skip to content

Add transitive access wideners for new Enchantment functional interfaces#5177

Merged
modmuss50 merged 3 commits intoFabricMC:26.1from
ThePotatoArchivist:patch-1
Feb 3, 2026
Merged

Add transitive access wideners for new Enchantment functional interfaces#5177
modmuss50 merged 3 commits intoFabricMC:26.1from
ThePotatoArchivist:patch-1

Conversation

@ThePotatoArchivist
Copy link
Contributor

At some point during the snapshot cycle Enchantment#applyEffects was modified to use the private functional interfaces GenericAction and FloatAction, which made it impossible to use the access-widened applyEffects without adding your own access wideners. This PR makes those functional interfaces public so applyEffects can be used properly.

@Patbox
Copy link
Member

Patbox commented Jan 30, 2026

You need to modify template.classtweaker and then run generateClassTweaker task with gradle

@ThePotatoArchivist
Copy link
Contributor Author

I think that's correct? I put it with the EnchantmentHelper interface wideners

@Patbox Patbox added enhancement New feature or request small change labels Jan 30, 2026
@modmuss50 modmuss50 added the status: merge me please Pull requests that are ready to merge label Feb 3, 2026
@modmuss50 modmuss50 merged commit 04ff198 into FabricMC:26.1 Feb 3, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request small change status: merge me please Pull requests that are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants