Skip to content

Lazy load examples command dependencies#47

Open
chrisx9z wants to merge 1 commit into
wevm:mainfrom
chrisx9z:fix/lazy-load-examples-command
Open

Lazy load examples command dependencies#47
chrisx9z wants to merge 1 commit into
wevm:mainfrom
chrisx9z:fix/lazy-load-examples-command

Conversation

@chrisx9z
Copy link
Copy Markdown

Summary

  • remove the top-level Examples import from CLI command registration
  • lazy-load the examples module only when build --check-examples or examples:check runs
  • keep zile new from loading the examples module, which imports typescript

Fixes #46.

Testing

  • git diff --check
  • rg -n "Examples|typescript" src/internal/cli/commands.ts src/Cli.ts src/Examples.ts

I could not run the Bun test suite locally because this environment does not have Bun installed.

Comment: Avoids importing the TypeScript-backed examples module when running commands such as zile new.
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.

zile new fails without typescript installed

1 participant