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

Add .scss files to upgrade script. #16285

Closed
wants to merge 3 commits into from

Conversation

GameLord2011
Copy link

I use an index.scss file in my project, and the upgrade script did not detect it.

I use an index.scss file in my project, and the upgrade script did not detect it.
@GameLord2011 GameLord2011 requested a review from a team as a code owner February 5, 2025 19:37
@GameLord2011
Copy link
Author

GameLord2011 commented Feb 5, 2025

WAIT DO NOT MERGE THIS YET, I NEED TO ADD MORE CODE!

@GameLord2011
Copy link
Author

Can anyone help with adding the scss file format (switch @import for @use if scss and whatnot)?

@wongjn
Copy link
Contributor

wongjn commented Feb 5, 2025

See #15716 (comment)

@philipp-spiess
Copy link
Member

Hey @GameLord2011. Thanks for the contribution but the comment that @wongjn linked to from my colleague @RobinMalfait is unfortunately still our stance on this.

Even if we fix the upgrade tool to work around some of the issues you already found (e.g. the @use stuff), SCSS is not something we support in Tailwind CSS v4: https://tailwindcss.com/docs/compatibility#sass-less-and-stylus. So the only way to make the upgrade tool work with SCSS files would be be to also convert all .scss files to .css which isn't even possible since not all SCSS features exist in Tailwind CSS.

Instead, our recommended migration approach is that you convert your SCSS project to CSS before you migrate to Tailwind CSS v4.

@GameLord2011
Copy link
Author

GameLord2011 commented Feb 6, 2025 via email

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.

3 participants