mcoms

Results 339 comments of mcoms

I'm going to hold off on this PR until after we make the spawning code less miserable.

No plans at this moment, so feel free!

The version command is from PaperAPI. It needs to be overridden with a new command for Glowstone. I don't want a patch for it in Glowkit because a implementation specific...

Perhaps the best way to handle this issue is to add a version provider interface to the API command class. Then, the server would register a version provider to the...

It would only be able to be set once, and would allow the basic command logic to be in the API, while having most of the implementation on the server.

I think I didn't explain something correctly. I am talking about the version command in Bukkit. A remote version check was added to the version command, which lives in Bukkit....

``` [ERROR] src/main/java/net/glowstone/entity/GlowLivingEntity.java:[77] (imports) CustomImportOrder: Wrong lexicographical order for 'org.bukkit.event.entity.EntityDamageByBlockEvent' import. Should be before 'org.bukkit.event.entity.EntityDamageByEntityEvent'. [ERROR] src/main/java/net/glowstone/entity/GlowLivingEntity.java:[1019] (coding) OverloadMethodsDeclarationOrder: Overload methods should not be split. Previous overloaded method located at...

I think @momothereal is working on supporting the legacy method as well. This issue can be a tracker for that.

This may have been fixed in https://github.com/GlowstoneMC/Glowstone/commit/5b8b93903b7ae255a6c6deaac7858292fd70faa5 with jansi being upgraded to 1.17.1.

Alright, we may have to wait until a new jansi comes out (as I see some progress on similar issues related to colors in the latest dev release), or for...