Skip to content

Conversation

@bhumika-ks31
Copy link

This PR fixes an issue in the SensorsBlocks module where duplicate methods in RGB blocks and input handling were causing redundant behavior.

Changes made:

  • Removed duplicate methods in GetRed, GetGreen, and GetBlue blocks.
  • Updated InputBlock handling for proper keyboard input flow.
  • Ensured pixel color detection works reliably without leaving ghost copies.
  • Cleaned up related updateParameter and arg methods for consistency.

All local tests pass successfully. These changes should make the pitch, loudness, and sensor blocks behave consistently and reduce unexpected duplicates or errors.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2025

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

SensorsBlocks.test.js

@github-actions
Copy link
Contributor

github-actions bot commented Nov 8, 2025

❌ Some Jest tests failed. Please check the logs and fix the issues before merging.

Failed Tests:

SensorsBlocks.test.js

Copy link

@zealot-zew zealot-zew 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 you have chat gpt the entire code file, thats why youre failing testcases

@walterbender
Copy link
Member

This PR has lots of unrelated changes, making it difficult to review.

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