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

feat: remove optimizer experimental and add preact examples #3712

Closed
wants to merge 0 commits into from
Closed

feat: remove optimizer experimental and add preact examples #3712

wants to merge 0 commits into from

Conversation

eryue0220
Copy link
Contributor

@stackblitz
Copy link

stackblitz bot commented Jul 1, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@netlify
Copy link

netlify bot commented Jul 1, 2023

Deploy Preview for fastidious-cascaron-4ded94 canceled.

Name Link
🔨 Latest commit b092985
🔍 Latest deploy log https://app.netlify.com/sites/fastidious-cascaron-4ded94/deploys/64c8b9ecda0b8d000887d4be

docs/config/index.md Outdated Show resolved Hide resolved
examples/preact/vitest.config.ts Outdated Show resolved Hide resolved
examples/preact-testing-lib/package.json Outdated Show resolved Hide resolved
globals: true,
environment: 'jsdom',
setupFiles: './test/setup.ts',
css: true,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is optimizer?

Again, the idea here is to show how to make aliasing react to preact work inside node_modules packages. In this example you need to rely on a package that has to alias react to preact. I already gave one example in one of my previous comments (react-router-dom).

@sheremet-va sheremet-va closed this Aug 1, 2023
@sheremet-va
Copy link
Member

sheremet-va commented Aug 1, 2023

I pushed changes, but it closed the PR, hm 🤔

Reopened in #3854

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.

Add more examples with enabled deps.optimizer
2 participants