-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Closed
Description
Currently (on the goog_module branch) the Blockly.utils.global sets exports.globalThis. As a result, its users invoke:
const {globalThis} = goog.require('Blockly.utils.global');
Since this util only does one thing, wouldn't it be better to set exports, allowing its users to invoke:
const globalThis = goog.require('Blockly.utils.global');
Simplifies the API. Not a breaking change as long as we make the change before the next release.