Vault

Vault

7M Downloads

vault-reload command in plugin.yml not implemented. Remove?

Psithief opened this issue ยท 3 comments

commented

If Vault needs no reload command, that's ok but it's still hanging around in plugin.yml and Vault.java.

commented

What?

Can you rephrase that? I don't understand.

An unimplemented vault-reload is not helpful. If you want a command to return your default command string, why not rename it vault-test? A reload command was brought up in issue #136, so finding there was a vault reload command that did nothing was confusing!

commented

ahh looks like it was intended to be implemented by cereal and I think he just never did. I thought it actually was implemented. Both of us getting confused now haha.

edit: fixed in latest v.

commented

it has the command to reload the methods just to have one, it works. You don't need to use it though and is often helpful for testing purposes. Why would I remove it?