Skip to content

feat: changed all Array.map(Array.range to Array.mapRange on all GetMatchGroupCopyPaste - #8081

Open
steve020607 wants to merge 1 commit into
mainfrom
steve23-patch4
Open

feat: changed all Array.map(Array.range to Array.mapRange on all GetMatchGroupCopyPaste#8081
steve020607 wants to merge 1 commit into
mainfrom
steve23-patch4

Conversation

@steve020607

Copy link
Copy Markdown
Collaborator

Summary

well pretty much explain by the title
changed all the usage of Array.map(Array.range to Array.mapRange on the GetMatchGroupCopyPaste/Wiki.lua

How did you test this change?

should be trivial enough though? i can try to test it

@steve020607
steve020607 requested review from a team as code owners September 11, 2026 09:25
@steve020607

Copy link
Copy Markdown
Collaborator Author

weird error, idk what went wrong

---@return table
function WikiCopyPaste.getStart(template, id, modus, args)
args.namedMatchParams = false
args.headersUpTop = Logic.readBool(Logic.emptyOr(args.headersUpTop, true))

@hjpalpha hjpalpha Sep 11, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

possible solution to shut up the anno warning:

	args.headersUpTop = Logic.nilOr(Logic.readBoolOrNil(args.headersUpTop), true)

@hjpalpha

Copy link
Copy Markdown
Collaborator

weird error, idk what went wrong

that check checks all annotations in changed files
so if there is a file with a bad annotation from before the PR that gets changed in the PR it flags those issues
the idea is to clean them all up over time :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants