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

ExperimentalSyntax #7575

Open
wants to merge 7 commits into
base: dev/feature
Choose a base branch
from

Conversation

TheAbsolutionism
Copy link
Contributor

@TheAbsolutionism TheAbsolutionism commented Feb 2, 2025

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 and Equippable 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

@sovdeeth sovdeeth added the enhancement Feature request, an issue about something that could be improved, or a PR improving something. label Feb 2, 2025
Copy link
Member

@Efnilite Efnilite left a 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

Copy link
Member

@cheeezburga cheeezburga left a 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.

@APickledWalrus
Copy link
Member

I think just ExperimentalSyntax could potentially work, but looks good either way.

I agree with this. I would also have it extend SyntaxElement

@TheAbsolutionism TheAbsolutionism changed the title ExperimentRestrictedSyntax ExperimentalSyntax Feb 4, 2025
@Pesekjak
Copy link
Contributor

This should use the Experiment class instead of the Feature implementation, so it's useful outside Skript's experiments.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Feature request, an issue about something that could be improved, or a PR improving something.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants