fix: Allow overriding `VariableModel` via `options.plugins`. by gonfunko · Pull Request #9257 · RaspberryPiFoundation/blockly
Navigation Menu
{{ message }}
RaspberryPiFoundation / blockly Public
- Notifications You must be signed in to change notification settings
- Fork 3.8k
Merged
fix: Allow overriding VariableModel via options.plugins.#9257
fix: Allow overriding VariableModel via options.plugins.#9257
Conversation
Copy link Copy Markdown
Contributor
gonfunko
commented
Jul 28, 2025
gonfunko
commented
The basics
The details
Resolves
Fixes #9256
Proposed Changes
This PR updates VariableMap to retrieve the VariableModel class to be used based on the injection options.plugins. Previously, this could only be overridden by manually registering an alternative implementation with the registry as the default implementation.
gonfunko
requested a review
from a team
as a code owner
gonfunko
requested a review
from rachel-fenichel
github-actions
bot
assigned
rachel-fenichel
gonfunko
force-pushed
the
inject-variable-model
branch
from
4486fb5 to
39dcb45
Compare
rachel-fenichel approved these changes Jul 30, 2025
gonfunko
deleted the
inject-variable-model
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment