Skip to content

remove addition of underscores to variables #502

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 8, 2025

Conversation

PaulaScharf
Copy link
Member

When getting a variable an extra underscore is added to underscores, but not when setting it. Declaring and using a variable which is for example called "player_1" doesnt work at the moment.

The underscores used to be added to deal with special characters, but that is handled differently now.

Copy link

github-actions bot commented Jul 6, 2025

Tests

Test Name Status Flaky Duration
Blockly Editor Page Tests [Blockly] visits the editor page 979ms
Blockly Editor Page Tests [Blockly] visits the tutorial page 164ms
Blockly Editor Page Tests [Blockly] visits the gallery page 123ms
Blockly Editor Page Tests [Blockly] visits the faq page 122ms
Blockly Editor Page Tests [Blockly] visits the settings page 119ms
Blockly Editor Page Tests [Blockly] visits the login page 117ms
Blockly Editor Page Tests [Blockly] navigates to tutorial and back 11.9s
Blockly Editor Page Tests [Blockly] selects senseBox ESP 1.9s
Blockly Editor Page Tests [Blockly] selects senseBox MCU 2.6s
Blockly Editor Page Tests [Blockly] selects senseBox Mini 2.6s
Blockly Editor Page Tests [Blockly] compiles code 9.8s
Code Editor Page Tests [CodeEditor] visits the code editor page 1.4s
Code Editor Page Tests [CodeEditor] selects senseBox ESP 477ms
Code Editor Page Tests [CodeEditor] compiles code 603ms
Code Editor Page Tests [CodeEditor] opens reset dialog 600ms
Code Editor Page Tests [CodeEditor] fetches libraries 1.7s

Github Test Reporter by CTRF 💚

@Thiemann96 Thiemann96 merged commit c329ab2 into development Jul 8, 2025
5 checks passed
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