Skip to content

Conversation

@CrebsTheCoder
Copy link
Contributor

@CrebsTheCoder CrebsTheCoder commented Oct 17, 2025

Problem

There's a duplicate of [the] in the pattern for the tablisted players expression.

Solution

removed the [the] when the pattern is being registered

Testing Completed

After changes i compiled it and ran it in a test server and run my docs tool and voila no more dupes
image

Supporting Information


Completes: none
Related: none

@CrebsTheCoder CrebsTheCoder requested a review from a team as a code owner October 17, 2025 16:16
@CrebsTheCoder CrebsTheCoder requested review from Pesekjak and TheMug06 and removed request for a team October 17, 2025 16:16
@CrebsTheCoder
Copy link
Contributor Author

Note you can check the pattern duplicate on
SkriptLang Docs or SkDocs

@Burbulinis
Copy link
Contributor

@BaeFell?

@sovdeeth
Copy link
Member

Please include the name of the actual syntax element in the pr next time lol
you can't tell what this is about until you look at the code

@sovdeeth sovdeeth changed the title Syntax pattern had a duplicate [the] like [the] [the] Tablisted Players expression has a duplicate [the] like [the] [the] Oct 17, 2025
@sovdeeth sovdeeth added bug An issue that needs to be fixed. Alternatively, a PR fixing an issue. documentation Related to Skript's official documentation. labels Oct 17, 2025
@skriptlang-automation skriptlang-automation bot added the needs reviews A PR that needs additional reviews label Oct 17, 2025
@skriptlang-automation skriptlang-automation bot added feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. and removed needs reviews A PR that needs additional reviews labels Oct 17, 2025
@CrebsTheCoder
Copy link
Contributor Author

yeah mb i forgot to mention it

@sovdeeth sovdeeth changed the base branch from dev/feature to dev/patch October 18, 2025 02:04
@skriptlang-automation skriptlang-automation bot added patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. and removed feature-ready A PR/issue that has been approved, tested and can be merged/closed in the next feature version. labels Oct 18, 2025
@sovdeeth sovdeeth moved this to Awaiting Merge in 2.13 Releases Oct 18, 2025
@APickledWalrus APickledWalrus merged commit 6fddc35 into SkriptLang:dev/patch Nov 1, 2025
5 checks passed
@skriptlang-automation skriptlang-automation bot added the completed The issue has been fully resolved and the change will be in the next Skript update. label Nov 1, 2025
@github-project-automation github-project-automation bot moved this from Awaiting Merge to Done - Awaiting Release in 2.13 Releases Nov 1, 2025
@skriptlang-automation skriptlang-automation bot removed the patch-ready A PR/issue that has been approved and is ready to be merged/closed for the next patch version. label Nov 1, 2025
@APickledWalrus APickledWalrus moved this from Done - Awaiting Release to Done - Released in 2.13 Releases Nov 2, 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. completed The issue has been fully resolved and the change will be in the next Skript update. documentation Related to Skript's official documentation.

Projects

Status: Done - Released

Development

Successfully merging this pull request may close these issues.

6 participants