-
-
Notifications
You must be signed in to change notification settings - Fork 386
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
ExperimentalSyntax #7575
base: dev/feature
Are you sure you want to change the base?
ExperimentalSyntax #7575
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm with fuse's changes
src/main/java/ch/njol/skript/lang/ExperimentRestrictedSyntax.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think just ExperimentalSyntax
could potentially work, but looks good either way.
I agree with this. I would also have it extend SyntaxElement |
This should use the Experiment class instead of the Feature implementation, so it's useful outside Skript's experiments. |
Description
This PR aims to add a new interface that syntax elements can implement to state that the element requires an experiment to be enabled in order to use it.
Also gives a proper message on why the line is erroring and informs users on how to use it.
I plan on picking back up my
Tool Components
andEquippable Components
PRs making them experimental. Since it seems that they may not ever come out of experimental with Minecraft adding stuff almost each version. So this would help with that.Target Minecraft Versions: any
Requirements: none
Related Issues: none