Skip to content

fix: remove Babel build warning in seo checks#1187

Open
immanuwell wants to merge 1 commit into
wasmCloud:mainfrom
immanuwell:fix/seo-checks-type-imports
Open

fix: remove Babel build warning in seo checks#1187
immanuwell wants to merge 1 commit into
wasmCloud:mainfrom
immanuwell:fix/seo-checks-type-imports

Conversation

@immanuwell
Copy link
Copy Markdown
Contributor

Tiny cleanup but it hits every build, kinda noisy.

On main, npm run build prints the DefaultCheckOptions Babel scope tracker warning twice from the local seo checks package.
This swaps type-only imports to import type, so Babel stops treating that type like runtime code. Build output is clean again.

Repro

  1. npm ci
  2. npm run build
  3. See the DefaultCheckOptions warning twice

Checks

  1. Reverted this patch locally and reproduced the warning
  2. Re-applied it and ran npm run build again
  3. Warning is gone, build still passes

Signed-off-by: immanuwell <pchpr.00@list.ru>
@immanuwell immanuwell requested a review from a team as a code owner June 7, 2026 08:27
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 7, 2026

Deploy Preview for dreamy-golick-5f201e ready!

Name Link
🔨 Latest commit ef8acb9
🔍 Latest deploy log https://app.netlify.com/projects/dreamy-golick-5f201e/deploys/6a252b839ede85000881667a
😎 Deploy Preview https://deploy-preview-1187--dreamy-golick-5f201e.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

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