Skip to content

Fix variable name clashes#1537

Merged
cocomarine merged 9 commits into
mainfrom
499-fix-variable-and-import-caching
Jul 16, 2026
Merged

Fix variable name clashes#1537
cocomarine merged 9 commits into
mainfrom
499-fix-variable-and-import-caching

Conversation

@cocomarine

@cocomarine cocomarine commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Closes https://github.com/RaspberryPiFoundation/digital-editor-issues/issues/499

  • Turned out caching clearing was working fine, but there was variable name clash.
  • When printed out the variable name of the example code, it was assigned to sys, indicating it was getting mixed up with the global variable name used in the runPythonAsync .
Screenshot 2026-07-16 at 11 14 41
  • Updated the variables and now it shows the expected error.
Screenshot 2026-07-16 at 11 17 29

@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 13, 2026 16:09 — with GitHub Actions Inactive
@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 16, 2026 08:08 — with GitHub Actions Inactive
…berryPiFoundation/editor-ui into 499-fix-variable-and-import-caching
@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 16, 2026 08:32 — with GitHub Actions Inactive
@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 16, 2026 08:56 — with GitHub Actions Inactive
@cocomarine cocomarine changed the title Fix variable name clashes and import issues Fix variable name clashes Jul 16, 2026
@cocomarine
cocomarine marked this pull request as ready for review July 16, 2026 10:07
@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 16, 2026 10:18 — with GitHub Actions Inactive
Comment thread cypress/e2e/spec-wc-pyodide.cy.js Outdated

@zetter-rpf zetter-rpf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice one, the cause wasn't obvious to me so thanks for getting to the bottom of it.

@cocomarine
cocomarine temporarily deployed to previews/1537/merge July 16, 2026 13:42 — with GitHub Actions Inactive
@cocomarine
cocomarine merged commit f671f89 into main Jul 16, 2026
8 checks passed
@cocomarine
cocomarine deleted the 499-fix-variable-and-import-caching branch July 16, 2026 15:26
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