Skip to content

Allow plural types in docs.json#7698

Merged
APickledWalrus merged 10 commits into
dev/patchfrom
patch/event-values-name
May 1, 2025
Merged

Allow plural types in docs.json#7698
APickledWalrus merged 10 commits into
dev/patchfrom
patch/event-values-name

Conversation

@Efnilite
Copy link
Copy Markdown
Member

Description

Adds displaying texts when the type is String[], where it would have previously not displayed at all.


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

@Efnilite Efnilite added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. documentation Related to Skript's official documentation. labels Mar 13, 2025
Comment thread src/main/java/ch/njol/skript/doc/JSONGenerator.java Outdated
@Efnilite Efnilite requested a review from Fusezion March 13, 2025 15:23
Comment thread src/main/java/ch/njol/skript/doc/JSONGenerator.java Outdated
@Efnilite Efnilite requested a review from sovdeeth April 1, 2025 15:32
@Efnilite Efnilite requested a review from a team as a code owner April 1, 2025 15:32
Copy link
Copy Markdown
Member

@sovdeeth sovdeeth left a comment

Choose a reason for hiding this comment

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

looks like conflicts were improperly resolved

@Efnilite Efnilite requested a review from sovdeeth April 9, 2025 09:30
@Burbulinis Burbulinis added the patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. label May 1, 2025
@APickledWalrus APickledWalrus merged commit 10bfad9 into dev/patch May 1, 2025
5 checks passed
@APickledWalrus APickledWalrus deleted the patch/event-values-name branch May 1, 2025 16:35
erenkarakal pushed a commit to erenkarakal/Skript that referenced this pull request Nov 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. documentation Related to Skript's official documentation. patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants