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

fix!: change default value of css to false, align with webpack #6910

Merged
merged 1 commit into from
Jun 26, 2024

Conversation

JSerFeng
Copy link
Collaborator

@JSerFeng JSerFeng commented Jun 25, 2024

Summary

Breaking Change

Align default value of css with webpack, the default value is false now.

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@JSerFeng JSerFeng requested a review from ahabhgk June 25, 2024 08:16
@github-actions github-actions bot added the team The issue/pr is created by the member of Rspack. label Jun 25, 2024
Copy link

netlify bot commented Jun 25, 2024

Deploy Preview for rspack canceled.

Name Link
🔨 Latest commit 4fff03a
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/667bb6a6fa5a6a00081f7452

@ahabhgk ahabhgk requested a review from LingyuCoder June 25, 2024 08:18
LingyuCoder
LingyuCoder previously approved these changes Jun 25, 2024
@JSerFeng JSerFeng enabled auto-merge (squash) June 25, 2024 08:28
@JSerFeng JSerFeng force-pushed the refactor/some-defaults branch 3 times, most recently from ccbc07f to a45c1e9 Compare June 25, 2024 12:09
@JSerFeng JSerFeng force-pushed the refactor/some-defaults branch 2 times, most recently from fa971ca to c546b95 Compare June 26, 2024 03:27
@hardfist
Copy link
Contributor

we should update the document about experiments.css usage at the same time

@hardfist
Copy link
Contributor

this title is misleading, it should be feat or fix and the description should contain breaking change notice

@JSerFeng JSerFeng changed the title chore: switch default value of css to false fix: align default value of css to false Jun 26, 2024
@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Jun 26, 2024
@ahabhgk ahabhgk changed the title fix: align default value of css to false fix!: align default value of css to false Jun 26, 2024
@JSerFeng JSerFeng changed the title fix!: align default value of css to false fix!: change default value of css to false, align with webpack Jun 26, 2024
@JSerFeng JSerFeng force-pushed the refactor/some-defaults branch 2 times, most recently from cce5486 to 1310a99 Compare June 26, 2024 06:20
@JSerFeng JSerFeng disabled auto-merge June 26, 2024 07:08
@JSerFeng
Copy link
Collaborator Author

JSerFeng commented Jun 26, 2024

we should update the document about experiments.css usage at the same time

I'll add docs asap

@JSerFeng JSerFeng merged commit 62b396a into main Jun 26, 2024
29 checks passed
@JSerFeng JSerFeng deleted the refactor/some-defaults branch June 26, 2024 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: bug fix release: bug related release(mr only) team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants