Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removing or renaming /test command #1079

Closed
altrisi opened this issue Aug 24, 2021 · 2 comments
Closed

Removing or renaming /test command #1079

altrisi opened this issue Aug 24, 2021 · 2 comments

Comments

@altrisi
Copy link
Collaborator

altrisi commented Aug 24, 2021

It is actually used by the vanilla game testing suite, and fabric api plans to make it available in one of the next versions (FabricMC/fabric#1622), where Carpet could at some point use it (either the test suite or the api), rendering the command useless since it would have conflicts.

@James103
Copy link
Contributor

fabric api plans to make it available in one of the next versions (FabricMC/fabric#162),

Did you mean FabricMC/fabric#1622 instead?

@modmuss50
Copy link

Just a note on this, for now we are only going to be enabling it in dev envs, fabric's game test module will not be shiped in the fat jar that most players use. Its still possible to use outside of a dev env if installed manually however.

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

No branches or pull requests

3 participants