Skip to content

chore: remove circular dependency between blockly and dev-tools - #10207

Merged
lizschwab merged 1 commit into
RaspberryPiFoundation:mainfrom
lizschwab:5834
Jul 28, 2026
Merged

chore: remove circular dependency between blockly and dev-tools#10207
lizschwab merged 1 commit into
RaspberryPiFoundation:mainfrom
lizschwab:5834

Conversation

@lizschwab

Copy link
Copy Markdown
Contributor

The basics

The details

Resolves

Fixes

Proposed Changes

  • Removes the Advanced Playground link from the playground
  • Removes the Advanced Playground html file from the Blockly package
  • Removes the magic_symlink script, which was only used for running the Advanced Playground directly in Blockly
  • Removes dependency on @blockly/dev-tools and @blockly/theme-modern from blockly package (theme-modern is only needed for running the Advanced Playground)

Reason for Changes

We need to resolve the circular dependency to get Lerna and Nx to be able to automate build processes. As things are now, they cannot resolve build order because of the circular dependency.

Additional Information

This will change the workflow for running the advanced playground.

New Advanced Playground workflow:

  1. Run npm run start from packages/plugins/dev-tools/
    - or -
    Run npm run start --workspace=@blockly/dev-tools from monorepo root
  2. Make changes in blockly
  3. From a separate terminal window, run npm run build from packages/blockly
    - or -
    Run npm run build --workspace=blockly from monorepo root

@lizschwab
lizschwab requested a review from a team as a code owner July 28, 2026 22:18
@lizschwab
lizschwab requested a review from gonfunko July 28, 2026 22:18
@github-actions github-actions Bot added the PR: chore General chores (dependencies, typos, etc) label Jul 28, 2026
@lizschwab
lizschwab merged commit 0d90384 into RaspberryPiFoundation:main Jul 28, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: chore General chores (dependencies, typos, etc)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants