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

fix(react): safer experimentalReactChildren communication #8651

Closed
wants to merge 1 commit into from

Conversation

lilnasy
Copy link
Contributor

@lilnasy lilnasy commented Sep 23, 2023

Changes

Testing

Existing tests should pass.

Docs

Does not affect usage.

@changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2023

🦋 Changeset detected

Latest commit: 3378f24

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added pkg: react Related to React (scope) pkg: integration Related to any renderer integration (scope) labels Sep 23, 2023
@lilnasy
Copy link
Contributor Author

lilnasy commented Sep 23, 2023

Odd that it only manifests when using the deno adapter. More odd that it works fine with pnpm link.

A similar issue affects the vue integration #8390 (comment). Failing plugin: integrations/vue/src/index.ts.

Draft until monday.

@lilnasy
Copy link
Contributor Author

lilnasy commented Sep 23, 2023

Closing in favor of #8652.

@lilnasy lilnasy closed this Sep 23, 2023
@lilnasy lilnasy deleted the fix-8390 branch September 23, 2023 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: integration Related to any renderer integration (scope) pkg: react Related to React (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Astro 3 - build failed with Deno adapter and React integration
1 participant