Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Use IVariableMap instead of VariableMap #8371

Closed
1 task done
gonfunko opened this issue Jul 15, 2024 · 0 comments
Closed
1 task done

Use IVariableMap instead of VariableMap #8371

gonfunko opened this issue Jul 15, 2024 · 0 comments
Assignees
@gonfunko
Labels
issue: bug Describes why the code or behaviour is wrong issue: triage Issues awaiting triage by a Blockly team member
Milestone
v12

Comments

@gonfunko
Copy link
Contributor

Check for duplicates

  • I have searched for similar issues before opening a new one.

Description

#8369 introduced a new IVariableMap interface. Core should be updated to use this rather than hardcoding VariableMap, and to only reference the formalized API.

Reproduction steps

No response

Stack trace

No response

Screenshots

No response

Browsers

No response
@gonfunko gonfunko added issue: bug Describes why the code or behaviour is wrong issue: triage Issues awaiting triage by a Blockly team member labels Jul 15, 2024
@gonfunko gonfunko added this to the v12 milestone Jul 15, 2024
@gonfunko gonfunko self-assigned this Jul 15, 2024
@gonfunko gonfunko mentioned this issue Jul 19, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gonfunko gonfunko

Labels
issue: bug Describes why the code or behaviour is wrong issue: triage Issues awaiting triage by a Blockly team member
Projects
None yet
Milestone
v12
Development

No branches or pull requests
1 participant
@gonfunko