Skip to content

Conversation

@modmuss50
Copy link
Member

Also means we arent backwards compatible with 1.21.6, not the end of the world.

@Patbox
Copy link
Member

Patbox commented Jun 25, 2025

Couldn't it be added as optional?

@modmuss50
Copy link
Member Author

modmuss50 commented Jun 25, 2025

Couldn't it be added as optional?

I tried with addOptional either its broken or im stupid but it doesnt seem to make any difference.

@Patbox
Copy link
Member

Patbox commented Jun 25, 2025

Maybe using this.builder(ConventionalItemTags.MUSIC_DISCS).addOptional(Registries.ITEM.getKey(Items.MUSIC_DISC_LAVA_CHICKEN)) in seperate line?

@modmuss50
Copy link
Member Author

Maybe using this.builder(ConventionalItemTags.MUSIC_DISCS).addOptional(Registries.ITEM.getKey(Items.MUSIC_DISC_LAVA_CHICKEN)) in seperate line?

Same thing, looks like there is a bug somewhere....

@modmuss50 modmuss50 added the status: merge me please Pull requests that are ready to merge label Jun 27, 2025
@modmuss50
Copy link
Member Author

Going to merge this, there might be a bug here that is worth looking into. Thinking about it im not too keen on having to make code changes to support older versions, I dont think many people will care about .6 for long so its not a big loss.

@modmuss50 modmuss50 merged commit 3110759 into FabricMC:1.21.7 Jun 27, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status: merge me please Pull requests that are ready to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants