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

SecRunnable #7389

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

Conversation

TheAbsolutionism
Copy link
Contributor

Description

This PR allows users to create a type of Runnable by having code within the section being ran after waiting the provided timespan. Will not halt the script from running the code after the section.


Target Minecraft Versions: any
Requirements: none
Related Issues: none

Copy link
Contributor

@Fusezion Fusezion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not too much rn didn't have time working soon

src/main/java/ch/njol/skript/sections/SecRunnable.java Outdated Show resolved Hide resolved
@Pikachu920
Copy link
Member

personally, i don't think this should be compared to a runnable. it could be a handy feature (not sure how i feel about it yet), but it definitely isn't and probably shouldnt't be documented as if it was comparable to java's runnable.

@Moderocky
Copy link
Member

Just a side note, if you wanted to PR actual runnables like Java, a basic syntax for it is already in Skript, available in the test environment. See ExprSecRunnable.

@Moderocky Moderocky added the up for debate When the decision is yet to be debated on the issue in question label Jan 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
up for debate When the decision is yet to be debated on the issue in question
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants