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

Support Vue JSX #4897

Merged
merged 8 commits into from
Sep 29, 2022
Merged

Support Vue JSX #4897

merged 8 commits into from
Sep 29, 2022

Conversation

bluwy
Copy link
Member

@bluwy bluwy commented Sep 28, 2022

Changes

Close #4450

Introduce jsx: boolean option for @astrojs/vue. If true, it would add two renderers for .vue and .jsx files.

I'm not sure if there's a way to combine that like preact compat does.

Testing

added a vue jsx test

Docs

Documented in the readme.

@bluwy bluwy requested a review from a team as a code owner September 28, 2022 10:10
@changeset-bot
Copy link

changeset-bot bot commented Sep 28, 2022

🦋 Changeset detected

Latest commit: 6aedbea

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: astro Related to the core `astro` package (scope) pkg: vue Related to Vue (scope) pkg: integration Related to any renderer integration (scope) labels Sep 28, 2022
@matthewp matthewp marked this pull request as draft September 28, 2022 13:59
@bluwy bluwy marked this pull request as ready for review September 28, 2022 14:41
@bluwy
Copy link
Member Author

bluwy commented Sep 28, 2022

Fixed the intellisense issue. Good to go once I get docs approval!

@Jutanium
Copy link
Contributor

I think it would be good to show an example of passing options instead of a boolean, as the linked docs are slightly different (there the options are passed directly to the plugin, whereas here they're passed to the jsx: property, if I'm understanding correctly). Other than that and my couple comments, LGTM!

@bluwy
Copy link
Member Author

bluwy commented Sep 28, 2022

Added an example to show passing options! 766ade7

@bluwy bluwy merged commit fd9d323 into main Sep 29, 2022
@bluwy bluwy deleted the vue-jsx branch September 29, 2022 03:25
@astrobot-houston astrobot-houston mentioned this pull request Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pkg: astro Related to the core `astro` package (scope) pkg: integration Related to any renderer integration (scope) pkg: vue Related to Vue (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: not supporting vue tsx
3 participants