Skip to content

Fix wrong argument check for 'redstone set #' placeholder#4595

Merged
jurrejelle merged 1 commit intoGregTechCEu:1.20.1from
FyreDrakon:patch-1
Feb 10, 2026
Merged

Fix wrong argument check for 'redstone set #' placeholder#4595
jurrejelle merged 1 commit intoGregTechCEu:1.20.1from
FyreDrakon:patch-1

Conversation

@FyreDrakon
Copy link
Copy Markdown
Contributor

Changes argument check to allow 'redstone set' to work

What

'redstone set #' placeholder does not work; issue is due to checking second argument for 'set' when it should be used for power in line 298 (reference line 292 for correct implementation of 'redstone get')

How Was This Tested

This was not tested, as I do not know how to actually rebuild java projects or anything; I am just aiming to correct what seems to be a simple typo that breaks important computer monitor functionality.

Changes argument check to allow 'redstone set' to work
@FyreDrakon FyreDrakon requested a review from a team as a code owner February 9, 2026 22:26
@github-actions github-actions bot added the 1.20.1 label Feb 9, 2026
@jurrejelle jurrejelle added type: bugfix General bug fixes Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. labels Feb 10, 2026
@jurrejelle jurrejelle merged commit feb7643 into GregTechCEu:1.20.1 Feb 10, 2026
5 checks passed
@FyreDrakon FyreDrakon deleted the patch-1 branch February 10, 2026 20:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.20.1 Release: Patch - 0.0.X Smaller changes that either are bug fixes or very minor tweaks. type: bugfix General bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants