Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed browser-only usage of Emmett package introduced in 0.20.0 #125

Merged
merged 3 commits into from
Oct 25, 2024

Conversation

oskardudycz
Copy link
Collaborator

@oskardudycz oskardudycz commented Oct 25, 2024

Made plugin code unrelated to commander package. Thanks to that even if Emmett has commander peer dependency, it'll be only loaded for Command Line usage (when @event-driven-io/emmett/cli is imported.

Added test to detect future regression. See:

Fixes #124
Fixes #74

@oskardudycz oskardudycz added this to the 0.20.2 milestone Oct 25, 2024
@oskardudycz oskardudycz force-pushed the browser_compatibility_commander branch 3 times, most recently from fb54aa4 to 960835c Compare October 25, 2024 11:31
@oskardudycz oskardudycz force-pushed the browser_compatibility_commander branch from 960835c to 6ff3ab6 Compare October 25, 2024 11:47
@oskardudycz oskardudycz force-pushed the browser_compatibility_commander branch from e89c473 to d8f66ab Compare October 25, 2024 11:54
@oskardudycz oskardudycz merged commit d86351d into main Oct 25, 2024
3 checks passed
@oskardudycz oskardudycz deleted the browser_compatibility_commander branch October 25, 2024 11:56
@oskardudycz oskardudycz changed the title Removed reference to commander in a regular (non-CLI) package usage Fixed browser-only usage of Emmett package introduced in 0.20.0 Oct 25, 2024
@dilgerma
Copy link

thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Browser Only Add tests for Emmett ensuring browser compatibility
2 participants