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

[Bug]: if I don’t set minify to true, the code after building is not compressed #626

Closed
wjw-gavin opened this issue Dec 27, 2024 · 8 comments · Fixed by #651
Closed

[Bug]: if I don’t set minify to true, the code after building is not compressed #626

wjw-gavin opened this issue Dec 27, 2024 · 8 comments · Fixed by #651

Comments

@wjw-gavin
Copy link

wjw-gavin commented Dec 27, 2024

Version

"@rslib/core": "^0.2.2"

Details

Image Image Image

The default setting in the docs is true, but I found that if I don’t set minify: true, the code after building is not compressed. Please check this.

Reproduce link

https://github.com/web-infra-dev/rslib

Reproduce Steps

pnpm build

@9aoy
Copy link
Contributor

9aoy commented Dec 27, 2024

I think this is an issue with the rslib documentation, the minify default value of rslib is different from that of rsbuild.
@Timeless0911 hi, it looks like there are some rslib and rsbuild configurations that have different default values, but it's not pointed out in the rslib documentation.

@wjw-gavin
Copy link
Author

I think this is an issue with the rslib documentation, the minify default value of rslib is different from that of rsbuild. @Timeless0911 hi, it looks like there are some rslib and rsbuild configurations that have different default values, but it's not pointed out in the rslib documentation.

Sorry, I didn't pay attention to the repository. I opened github from the rslib document. I don't know why it jumped to rsbuild, so I asked the question directly.

@Timeless0911
Copy link
Contributor

The default setting in the docs is true

@wjw-gavin where is this from?

@Timeless0911 Timeless0911 transferred this issue from web-infra-dev/rsbuild Dec 27, 2024
@wjw-gavin
Copy link
Author

The default setting in the docs is true

@wjw-gavin where is this from?

https://rsbuild.dev/config/output/minify

@wjw-gavin wjw-gavin reopened this Dec 27, 2024
@wjw-gavin
Copy link
Author

wjw-gavin commented Dec 27, 2024

Image The configuration I found here, and then I jumped to the documentation of rsbuild. Then I found that the documentation was inconsistent with the actual results, so I clicked on the rsbuild repository address and asked a question.

@Timeless0911
Copy link
Contributor

In library development, minify is not a necessary option, which is very detrimental to the readability of the output.

The default value of Rslib is somewhere different with Rsbuild, we will declare it in Rslib documentation.

Thank you for your wise eyes to discover the shortcomings of Rslib documentation now.

@wjw-gavin
Copy link
Author

In library development, minify is not a necessary option, which is very detrimental to the readability of the output.

The default value of Rslib is somewhere different with Rsbuild, we will declare it in Rslib documentation.

Thank you for your wise eyes to discover the shortcomings of Rslib documentation now.

OK, The jump between the two documents makes me a little dizzy. 😂

@Timeless0911
Copy link
Contributor

I will improve that to introduce the difference between Rsbuild next week.

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 a pull request may close this issue.

3 participants