Skip to content

Format entire repo with Prettier - #726

Merged
lawsie merged 1 commit into
flipcomputing:mainfrom
lawsie:chore/prettier-format-repo
Jul 28, 2026
Merged

Format entire repo with Prettier#726
lawsie merged 1 commit into
flipcomputing:mainfrom
lawsie:chore/prettier-format-repo

Conversation

@lawsie

@lawsie lawsie commented Jul 28, 2026

Copy link
Copy Markdown
Collaborator

The prettier --check . failed on 215 files and the CI check was left non-blocking. This applies prettier --write . across the whole tree in one pass.

Formatting only — no behavioural change:

  • ASTs compared before/after for all 191 changed JS/MJS files: identical, except input/onScreenSource.js where quoted object keys became bare identifiers ("ArrowUp": -> ArrowUp:), which is semantically equivalent.
  • HTML tag inventories unchanged;
     content untouched.
  • Markdown trailing double-spaces removed only on list items and paragraph ends, where they are not hard line breaks.
  • eslint clean, test:input 242 passing, test:microbit 72 passing, vite build succeeds — all matching the pre-format baseline.

The repo had never been run through Prettier, so `prettier --check .`
failed on 215 files and the CI check was left non-blocking. This applies
`prettier --write .` across the whole tree in one pass.

Formatting only — no behavioural change:
- ASTs compared before/after for all 191 changed JS/MJS files: identical,
  except input/onScreenSource.js where quoted object keys became bare
  identifiers ("ArrowUp": -> ArrowUp:), which is semantically equivalent.
- HTML tag inventories unchanged; <pre> content untouched.
- Markdown trailing double-spaces removed only on list items and
  paragraph ends, where they are not hard line breaks.
- eslint clean, test:input 242 passing, test:microbit 72 passing,
  vite build succeeds — all matching the pre-format baseline.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@lawsie
lawsie merged commit 28ea856 into flipcomputing:main Jul 28, 2026
2 of 3 checks passed
@coderabbitai

coderabbitai Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Too many files!

This PR contains 215 files, which is 115 over the limit of 100.

To get a review, narrow the scope:
• coderabbit review --committed # exclude uncommitted changes
• coderabbit review --dir # limit to a subdirectory
• coderabbit review --base # compare against a closer base

Upgrade to a paid plan to raise the limit.

This review couldn't start because sufficient usage credits or metered capacity aren't available. Add credits or update usage-based reviews in the billing tab, then retry.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 73a70e42-e103-43da-967f-8bfffa921f64

📥 Commits

Reviewing files that changed from the base of the PR and between e3aac85 and 80bcc04.

📒 Files selected for processing (215)
  • .claude/skills/verify/SKILL.md
  • .github/ISSUE_TEMPLATE/bug_report.md
  • .github/ISSUE_TEMPLATE/feature_request.md
  • .github/workflows/run-mocha-tests.yml
  • .github/workflows/static.yml
  • AGENTS.md
  • AI_POLICY.md
  • API.md
  • CLAUDE.md
  • CONTRIBUTING.md
  • README.md
  • TRADEMARK.md
  • TRANSLATION.md
  • accessibility/accessibility.js
  • accessibility/uiA11y.js
  • api/animate.js
  • api/camera.js
  • api/control.js
  • api/csg.js
  • api/effects.js
  • api/events.js
  • api/material.js
  • api/mesh.js
  • api/microbit.js
  • api/models.js
  • api/movement.js
  • api/physics.js
  • api/sensing.js
  • api/sound.js
  • api/transform.js
  • api/xr.js
  • blocks/animate.js
  • blocks/base.js
  • blocks/blockIcons.js
  • blocks/blocks.js
  • blocks/camera.js
  • blocks/colour.js
  • blocks/colourvalidation.js
  • blocks/combine.js
  • blocks/condition.js
  • blocks/connect.js
  • blocks/control.js
  • blocks/effects.js
  • blocks/events.js
  • blocks/fieldMicrobitImage.js
  • blocks/materials.js
  • blocks/microbitImagePattern.js
  • blocks/models.js
  • blocks/physics.js
  • blocks/scene.js
  • blocks/sensing.js
  • blocks/shapes.js
  • blocks/sound.js
  • blocks/text.js
  • blocks/text_join_shadow.js
  • blocks/transform.js
  • blocks/xr.js
  • config.js
  • cubeart.html
  • dev-docs/API_QUALITY_TOOLS.md
  • dev-docs/API_RECONCILIATION_PLAN.md
  • docs/endless.md
  • docs/screen-reader.md
  • embed-example.html
  • eslint.config.mjs
  • flock.js
  • generators/generators-animate.js
  • generators/generators-condition.js
  • generators/generators-control.js
  • generators/generators-data.js
  • generators/generators-deprecated.js
  • generators/generators-events.js
  • generators/generators-functions.js
  • generators/generators-material.js
  • generators/generators-math.js
  • generators/generators-scene.js
  • generators/generators-sensing.js
  • generators/generators-sound.js
  • generators/generators-text.js
  • generators/generators-transform.js
  • generators/generators-utilities.js
  • generators/generators.js
  • generators/mesh-state.js
  • index.html
  • input/bindings.js
  • input/gamepadSource.js
  • input/inputManager.js
  • input/joystickSource.js
  • input/keyboardSource.js
  • input/normaliseKey.js
  • input/onScreenSource.js
  • input/xrSource.js
  • locale/de.js
  • locale/en.js
  • locale/es.js
  • locale/fr.js
  • locale/it.js
  • locale/pl.js
  • locale/pt.js
  • locale/sv.js
  • main/blockhandling.js
  • main/blocklyinit.js
  • main/context.js
  • main/customCommentIcon.js
  • main/debug.js
  • main/examples.js
  • main/export.js
  • main/files.js
  • main/loading.js
  • main/main.js
  • main/menu.js
  • main/themes.js
  • main/translation.js
  • main/view.js
  • microbit/manager.js
  • microbit/protocol.js
  • microbit/usbTransport.js
  • package.json
  • playwright.config.js
  • scripts/api-coverage-report.mjs
  • scripts/canvas-buffer-check.mjs
  • scripts/check-link-security.mjs
  • scripts/csp-smoke.mjs
  • scripts/run-api-tests.mjs
  • scripts/run-input-tests.mjs
  • scripts/run-microbit-tests.mjs
  • scripts/test-coverage-matrix.mjs
  • scripts/utils/extract-api-methods.mjs
  • scripts/utils/parse-api-md.mjs
  • scripts/utils/parse-jsdoc.mjs
  • scripts/utils/test-analyzer.mjs
  • style.css
  • tests/accessibility/getPlayerMesh.test.js
  • tests/accessibility/index.test.js
  • tests/animate.test.js
  • tests/babylon.test.js
  • tests/blocks.test.js
  • tests/boxes.test.js
  • tests/buttoncontrols.test.js
  • tests/camera.test.js
  • tests/characterAnimations.test.js
  • tests/colourvalidation.test.js
  • tests/concurrency.test.js
  • tests/control.test.js
  • tests/effects.test.js
  • tests/events.test.js
  • tests/generators-material.test.js
  • tests/getProperty.test.js
  • tests/glide.test.js
  • tests/glideToObject.test.js
  • tests/input/bindings.test.js
  • tests/input/cameraControls.test.js
  • tests/input/consumers.test.js
  • tests/input/gamepadSource.test.js
  • tests/input/index.test.js
  • tests/input/inputManager.test.js
  • tests/input/joystickSource.test.js
  • tests/input/keyboardSource.test.js
  • tests/input/onScreenSource.test.js
  • tests/input/xrSource.test.js
  • tests/materials.test.js
  • tests/math.test.js
  • tests/mesh-hierarchy.test.js
  • tests/microbit/field.test.js
  • tests/microbit/generators.test.js
  • tests/microbit/index.test.js
  • tests/microbit/manager.test.js
  • tests/microbit/protocol.test.js
  • tests/movement.test.js
  • tests/notifications.test.js
  • tests/objects.test.js
  • tests/onscreencontrols.test.js
  • tests/physics.test.js
  • tests/playwright/aria-input-labels.spec.js
  • tests/playwright/blocks.spec.js
  • tests/playwright/example-loading.spec.js
  • tests/playwright/flock.spec.js
  • tests/playwright/speak-language-options.spec.js
  • tests/printtext.test.js
  • tests/scene.test.js
  • tests/security/color-validation.test.js
  • tests/security/free-text-validation.test.js
  • tests/security/index.test.js
  • tests/sensing.test.js
  • tests/shapes.test.js
  • tests/sound-integration.test.js
  • tests/sound-verification.test.js
  • tests/sound.test.js
  • tests/sound2.test.js
  • tests/status.test.js
  • tests/tests.html
  • tests/transform.rotate.test.js
  • tests/transform.scale.test.js
  • tests/transform.translate.test.js
  • tests/ui/interactIndicator.test.js
  • tests/uitextbutton.test.js
  • tests/utils/audioTestUtils.js
  • tests/xr-export.test.js
  • tests/xr.test.js
  • toolbox.js
  • tutorial.js
  • ui/addmenu.js
  • ui/addmeshes.js
  • ui/axis-keyboard.js
  • ui/blocklyshadowutil.js
  • ui/blocklyutil.js
  • ui/blockmesh.js
  • ui/canvas-utils.js
  • ui/colourpicker.css
  • ui/colourpicker.js
  • ui/designview.js
  • ui/gizmos.js
  • ui/meshhelpers.js
  • ui/notifications.js
  • vite.config.js

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

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.

1 participant