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

console: Move to rspack/rsbuild #1414

Merged
merged 3 commits into from
Oct 11, 2024
Merged

Conversation

malinskibeniamin
Copy link
Contributor

@malinskibeniamin malinskibeniamin commented Aug 27, 2024

Similarly to other PRs, we would like to use rspack and stay consistent with the other Cloud UIs.

This PR paves the way towards enhanced module federation usage, and cloud UI PR is also being migrated rspack for that reason.

The build + start commands are already working.

Needs extensive checks (I didn't have console fully set up to try out locally e2e)

We also added rsdoctor plugin which is a tool for bundle analysis.
Screenshot 2024-08-28 at 09 53 05
Screenshot 2024-08-28 at 09 54 43

We also have monaco editor supported with YAML schema configurations and auto-complete.
Screenshot 2024-10-09 at 16 09 02
Screenshot 2024-10-09 at 16 09 14
Screenshot 2024-10-09 at 16 10 15


https://github.com/web-infra-dev/rspack/releases
https://rspack.dev/blog/announcing-1-0
https://github.com/rspack-contrib/performance-compare

https://x.com/rspack_dev
https://rspack.dev/guide/migration/cra
https://rspack.dev/guide/features/module-federation

Example build

npm run build

~/D/g/c/frontend (chore/move-console-to-rspack|✚2) $ npm run build

> [email protected] build
> rsbuild build

  Rsbuild v1.0.1-rc.0

● web ━━━━━━━━━━━━━━━━━━━━━━━━━ (100%) emitting after emit                                                                                                                                ready   Compiled in 10.9 s (web)
info    Production file sizes for web:

...

  Total size:  20274.7 kB
  Gzipped size:  8797.4 kB

npm run start

~/D/g/c/frontend (chore/move-console-to-rspack|✚2) $ npm run start

> [email protected] start
> rsbuild dev

  Rsbuild v1.0.1-rc.0

  > Local:    http://localhost:3000/
  > Network:  http://192.168.0.196:3000/

start   Compiling...
ready   Compiled in 2.56 s (web)
chore.move-console-to-rspack-console-standalone.mov

@CLAassistant
Copy link

CLAassistant commented Aug 27, 2024

CLA assistant check
All committers have signed the CLA.

@malinskibeniamin malinskibeniamin force-pushed the chore/move-console-to-rspack branch 2 times, most recently from d9af571 to ed980c6 Compare August 27, 2024 11:13
@malinskibeniamin malinskibeniamin added performance Performance is one of the concerns ui/ux labels Aug 27, 2024
@malinskibeniamin malinskibeniamin force-pushed the chore/move-console-to-rspack branch 2 times, most recently from d72e821 to e24a2ac Compare August 28, 2024 06:19
@malinskibeniamin malinskibeniamin changed the title console: Move to rspack console: Move to rspack 1.0.0 Aug 28, 2024
@malinskibeniamin malinskibeniamin changed the title console: Move to rspack 1.0.0 console: Move to rspack/rsbuild Sep 30, 2024
@sago2k8 sago2k8 force-pushed the chore/move-console-to-rspack branch 3 times, most recently from da8f5c5 to 96d79f7 Compare October 7, 2024 13:31
@malinskibeniamin malinskibeniamin force-pushed the chore/move-console-to-rspack branch 2 times, most recently from df371c0 to 0a5d9c8 Compare October 7, 2024 14:50
@sago2k8 sago2k8 force-pushed the chore/move-console-to-rspack branch 2 times, most recently from 0f15272 to 2360825 Compare October 8, 2024 13:29
@malinskibeniamin malinskibeniamin force-pushed the chore/move-console-to-rspack branch 5 times, most recently from 7b698a5 to 7e5e156 Compare October 10, 2024 20:22
@malinskibeniamin malinskibeniamin force-pushed the chore/move-console-to-rspack branch 3 times, most recently from 617f0f2 to f1643c3 Compare October 10, 2024 21:05
Copy link
Contributor

@sago2k8 sago2k8 left a comment

Choose a reason for hiding this comment

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

LGTM, only a minor comment

}
Copy link
Contributor

Choose a reason for hiding this comment

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

[nit]: could you make your editor add EOF line, so there are no formatting changes.

Copy link
Contributor Author

@malinskibeniamin malinskibeniamin Oct 11, 2024

Choose a reason for hiding this comment

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

I have "files.insertFinalNewline": true because there should be an EOF line always, now it's there (previously there wasn't any EOF character)

@malinskibeniamin malinskibeniamin merged commit 4bb77d3 into master Oct 11, 2024
6 checks passed
@malinskibeniamin malinskibeniamin deleted the chore/move-console-to-rspack branch October 11, 2024 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
frontend performance Performance is one of the concerns ui/ux
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants