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

Fix Resourcepack Generator #11

Merged
merged 12 commits into from
Nov 25, 2022
Merged

Fix Resourcepack Generator #11

merged 12 commits into from
Nov 25, 2022

Conversation

OpenBagTwo
Copy link
Owner

  • Begins work on Update README #5, completely fleshing out the README
  • Fixes numerous issues in the resource pack generator where the generator wouldn't actually... create valid resource packs 🤦

I'd love to blame this on a bug? in Minecraft 1.19.3's resource resolver but the truth it that it was pretty obvious that I never tested even once that the zip files were valid resourcepacks.

OpenBagTwo and others added 12 commits November 25, 2022 14:29
I think I'm looking at going to mkdocs
with all the bs I put into the license manipulation and spec system that NO ONE WILL EVER USE I didn't bother to test one of these RPs EVEN ONCE?!?
tbf, only most of this was my fault--it looks like Minecraft 1.19.3 is a lot stricter in terms of where it will look for textures.
@OpenBagTwo
Copy link
Owner Author

There may be some sense in upping the pack format when building on the 1.19.3 branch, but as long as it's possible to create a resource pack that is compatible with all supported versions of the game, I'd rather just have users deal with the warning than think they need to use a special version of the executable.

@OpenBagTwo OpenBagTwo merged commit 4efbf92 into 1.19 Nov 25, 2022
@OpenBagTwo OpenBagTwo deleted the writeme branch November 25, 2022 21:29
@OpenBagTwo OpenBagTwo restored the writeme branch November 25, 2022 21:29
@OpenBagTwo OpenBagTwo deleted the writeme branch November 25, 2022 21:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant