-
Notifications
You must be signed in to change notification settings - Fork 82
Migrate build config to rslib #1197
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
base: main
Are you sure you want to change the base?
Conversation
42ef5f4
to
1e6d2dd
Compare
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je met une review bloquante pour eviter que ce soit mergé avant d'etre discuté
import { defineConfig } from "@rslib/core"; | ||
import { join } from "path"; | ||
|
||
export default defineConfig({ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on ne pouvait pas faire du partage de config ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Je pense que oui mais comme on ne l'avait pas faire pour vitest et tsconfig je me suis dit que c'était pas utile ici.
postcssPresetEnv({ | ||
stage: 4, | ||
}), | ||
postcssNormalize, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pourquoi on supprime cssnano ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
on est sûr que rsbuild optimise mieux ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alors j'ai supprimé cssnano car rslib n'en tenais pas compte, du coup j'ai activé la minification de rslib mais j'ai pas regarder si l'opti été mieux ou pas pour moi elle semble équivalente.
Rslib utilise dernière Lightning CSS pour faire la minification du css qui est une lib plutôt réputer donc sa doit être bon.
soource:
https://rsbuild.dev/config/output/minify
https://rsbuild.dev/guide/styling/css-usage#lightning-css
https://lightningcss.dev/
https://www.npmjs.com/package/lightningcss
1e6d2dd
to
822ae5b
Compare
|
This PR updates the build process for both React and CSS packages:
tsc
torslib
.Advantage: Build files now use explicit import extensions.
postcss
torslib
(with the Sass plugin).Advantage: The rslib Sass plugin uses the modern API, so there are no more warnings.
This improves consistency and modernizes the build pipeline across the monorepo.