Skip to content

Modernize rgb() function with DefaultFunction.builder()#1

Open
Dev-Xeiji wants to merge 5 commits into
masterfrom
Modernize-rgb()-function-with-DefaultFunction.builder()
Open

Modernize rgb() function with DefaultFunction.builder()#1
Dev-Xeiji wants to merge 5 commits into
masterfrom
Modernize-rgb()-function-with-DefaultFunction.builder()

Conversation

@Dev-Xeiji
Copy link
Copy Markdown
Owner

Problem

SkriptLang#8543

Solution

  • Replace SimpleJavaFunction to DefaultFunction.builder()
  • Added Modifier.ranged(0, 255) for R/G/B/A parameters
  • Maintain identical behavior using Long.class and .intValue()

Testing Completed

Manually tested and verified that the issue at SkriptLang#8543 has been solved in this change, using the provided steps to reproduce the bug.

Supporting Information


Completes: SkriptLang#8543
Related: SkriptLang#8543
AI assistance: none

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.

3 participants