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(rsbuild): add react and vue support for app generation #29349

Merged
merged 14 commits into from
Dec 18, 2024

Conversation

Coly010
Copy link
Contributor

@Coly010 Coly010 commented Dec 13, 2024

Current Behavior

We do not have a generator that can scaffold a react or vue app using rsbuild.

Expected Behavior

Update the react application generator to support a bundler option of rsbuild
Update the vue application generator to support a bundler option of rsbuild

@Coly010 Coly010 self-assigned this Dec 13, 2024
Copy link

vercel bot commented Dec 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
nx-dev ✅ Ready (Inspect) Visit Preview Dec 18, 2024 4:24pm

Copy link

nx-cloud bot commented Dec 13, 2024

Your CI Pipeline Execution ↗ for commit 32bc425 ⏳ is in progress.

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ❌ Failed 29m, 43s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 1m, 2s View ↗
nx-cloud record -- nx format:check --base=0d666... ✅ Succeeded 19s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded 0s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded 17s View ↗
nx documentation --no-dte ✅ Succeeded 39s View ↗

Last updated at 2024-12-13 17:45:27UTC

Sent with 💌 from NxCloud.

Copy link

nx-cloud bot commented Dec 17, 2024

Your CI Pipeline Execution ↗ for commit 4638f8b is in progress ⏳

Command Status Duration Result
nx affected --targets=lint,test,build,e2e,e2e-c... ✅ Succeeded 9m 42s View ↗
nx run-many -t check-imports check-commit check... ✅ Succeeded 23s View ↗
nx-cloud record -- nx format:check --base=0d666... ✅ Succeeded 3s View ↗
nx-cloud record -- nx-cloud conformance:check ✅ Succeeded <1s View ↗
nx-cloud record -- nx sync:check ✅ Succeeded <1s View ↗
nx documentation --no-dte ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-18 16:31:23 UTC

@Coly010 Coly010 marked this pull request as ready for review December 17, 2024 12:44
@Coly010 Coly010 requested review from a team as code owners December 17, 2024 12:44
@Coly010 Coly010 force-pushed the rsbuild/react-support branch from bfafefd to 92a9766 Compare December 17, 2024 12:46
@Coly010 Coly010 force-pushed the rsbuild/react-support branch from 92a9766 to 9fc8ab5 Compare December 17, 2024 13:17
@Coly010 Coly010 requested review from a team, FrozenPandaz and vsavkin as code owners December 17, 2024 20:23
@Coly010 Coly010 changed the title feat(react): add bundler option for rsbuild feat(rsbuild): add react and vue support for app generation Dec 17, 2024
@Coly010 Coly010 force-pushed the rsbuild/react-support branch from 8bd92a0 to 4638f8b Compare December 18, 2024 16:15
);

// TODO: enable this when tests are passing again.
// Colum confirmed locally that the generated config and the playwright tests are working.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

image image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The other vue e2e test also has the runE2e block commented out.
We should investigate why it doesn't like CI later

@jaysoo jaysoo merged commit 4b586a1 into master Dec 18, 2024
6 checks passed
@jaysoo jaysoo deleted the rsbuild/react-support branch December 18, 2024 16:44
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.

2 participants