Skip to content
This repository has been archived by the owner on Apr 14, 2024. It is now read-only.

Misleading information in the readme #6

Open
fishermedders opened this issue May 6, 2023 · 0 comments
Open

Misleading information in the readme #6

fishermedders opened this issue May 6, 2023 · 0 comments

Comments

@fishermedders
Copy link

Since this project is ever-evolving, I figured that I'd pitch my two cents, so here goes.

When I was attempting to use TNT, the readme states that the anvil world and the .tnt file are to be named the same thing when converting an anvil world to TNT. I believe this to be misleading, as when you attempt this in practice, the it attempts to load the world (which would be an empty file), and throws an ArrayIndexOutOfBoundsException, which is the expected behavior (the file is empty, but exists, just no world to load from the .tnt file). It should be said in this area that on conversion, the .tnt file should not exist yet.

The reason I didn't submit a PR and chose issues instead was that I didn't want to decide the wording of the readme or the behavior of the library for you guys, but at the same time wanted to help out others who may stumble upon this in the future.

Also, yes, hindsight is 20/20 and this should have been obvious to me 😛

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant