Expose ArgumentReader#skip()

[MrSlimeDiamond]

Sponge | SpongeAPI

[aromaa]

This should follow the convention we already use for the other skip methods, so skipChar. You most likely also want to couple this with peekChar to have this to be more usably.

I'm also not opposed to just have a method to set the cursor position directly.

[MrSlimeDiamond]

It looks like there's already an ArgumentReader#peekCharacter method, but Mutable's methods use a char suffix instead of character. (for example parseChar). Perhaps this should be made consistent?

I'm happy to rename it to skipChar, but this means I'd need to open a PR to Sponge

[aromaa]

Sigh, there's also peekString inside the Mutable instead of being in the base interface which I was glancing at for other similar functions. We can't accept breaking changes in 12 so that shouldn't be done here. Let's just rename it to skipChar to keep it consistent with the Mutable inteface.

[MrSlimeDiamond]

@aromaa Updated, there's now a Sponge PR to implement this