SirSmurfy2
SirSmurfy2
> I'm not sure this should be based off of the doublevalue.toString at all. What do you suggest? I was originally going to make it: ``` "[the] number of decimal...
> Alternatively, you can forgo a toString entirely and instead repeatedly divide by 10 until x % 1 is within epsilon of 0/1. So something like this? ```java for (Number...
> also tests would be nice I genuinely tried to create a test for it, but I couldn't get it to work. If you look at the `Starter Commit` you...
> (I also can't see what changes you've requested, or at least I can't find them) He didn't do any suggestions, he just marked his review as requested changes with...
I just realized someone already made a PR.... welp I didnt see the "PR Available" on the issue
Here soon, gonna rewrite some things, like getting multiple beacons provided with multiple blocks, and setting data of multiple beacons instead of a singular
> Rather than all the reflection, that will be removed in a couple years, why not just only register the CowData class if the variant exists? (same for simple data,...
> why would it require reflection? I'm not entirely sure of how it works. But for some reason `Cow.class` was returning/redirecting to `AbstractCow.class` and within `#set(Cow cow)` it was throwing...
Please note, to optimally fix the `databaseName` and the `databaseType` field added through this PR, by simply setting within the constructor, will be a "breaking change" - LimeGlass
Question, should we also restrict the usage of same database names?