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

New Expression: Loaded Chunks of World #5662

Merged

Conversation

NotSoDelayed
Copy link
Contributor

Description

This PR adds support of getting a list of loaded chunks of a world into Skript!


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

@AyhamAl-Ali AyhamAl-Ali added the feature Pull request adding a new feature. label Apr 30, 2023
Copy link
Member

@AyhamAl-Ali AyhamAl-Ali left a comment

Choose a reason for hiding this comment

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

Nice PR ⚡

Copy link
Collaborator

@TheLimeGlass TheLimeGlass left a comment

Choose a reason for hiding this comment

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

Should probably be included within https://github.com/SkriptLang/Skript/blob/master/src/main/java/ch/njol/skript/expressions/ExprChunk.java with separate syntax pattern (all of the|all [the]) [loaded] chunks of %worlds% and [the] chunk[s] (of|%-directions%) %locations%

@NotSoDelayed
Copy link
Contributor Author

Aware that tests are failing.

@NotSoDelayed
Copy link
Contributor Author

Fixed cause of test failing and PR ready for review!

@Moderocky
Copy link
Member

This could have a test too, since spawn chunks ought to be loaded in the test environment.

@TheLimeGlass
Copy link
Collaborator

This could have a test too, since spawn chunks ought to be loaded in the test environment.

too many variables

@TheLimeGlass TheLimeGlass merged commit 996b57e into SkriptLang:master Jul 31, 2023
4 checks passed
@NotSoDelayed NotSoDelayed deleted the feature/expr-loaded-chunks branch August 1, 2023 04:59
Moderocky pushed a commit to Moderocky/Skript that referenced this pull request Sep 16, 2023
* Initial

* Support plural

* Merge with ExprChunk

* Fix

* Fix 2

* Cleanup class to obey code convention

* Requested changes

* Fix indent

* Indent docs and improved pattern

---------

Co-authored-by: LimeGlass <[email protected]>
NotSoDelayed added a commit to NotSoDelayed/Skript that referenced this pull request Oct 30, 2023
* Initial

* Support plural

* Merge with ExprChunk

* Fix

* Fix 2

* Cleanup class to obey code convention

* Requested changes

* Fix indent

* Indent docs and improved pattern

---------

Co-authored-by: LimeGlass <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Pull request adding a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants