Skip to content

Fix sass @import deprecation warnings#2360

Merged
entrotech merged 2 commits into
developfrom
2359-remove-import-rule
Dec 4, 2025
Merged

Fix sass @import deprecation warnings#2360
entrotech merged 2 commits into
developfrom
2359-remove-import-rule

Conversation

@fyliu

@fyliu fyliu commented Apr 10, 2025

Copy link
Copy Markdown
Member

What changes did you make?

  • migrated sass @import rules to @use
    • I actually used the sass-migrator package to help with the migration.
      npm install sass-migrator --legacy-peer-deps
      npx sass-migrator module --migrate-deps src/styles/App.scss
      
      After fixing a few initial problems, the tool did the rest.

Why did you make the changes (we will use this info to test)?

  • The @import rule is deprecated and will be removed in a future release

Testing

  • I'm not sure how to get the warnings to display yet
  • But otherwise, the imported variables like background color should still work after this change.

@fyliu

fyliu commented Apr 17, 2025

Copy link
Copy Markdown
Member Author

@fyliu replicate the conditions again and see if this fix is something that'll allow us to upgrade to a higher version of sass. This fix could be useful if it does.

@fyliu

fyliu commented Apr 17, 2025

Copy link
Copy Markdown
Member Author

I'd like to save this until after I'm finished dockerizing the project.

@entrotech entrotech changed the title Fix sass @import deprecation warings Fix sass @import deprecation warnings Dec 4, 2025
@entrotech
entrotech marked this pull request as ready for review December 4, 2025 03:24
@entrotech
entrotech merged commit bbfe7f3 into develop Dec 4, 2025
7 checks passed
@entrotech
entrotech deleted the 2359-remove-import-rule branch December 4, 2025 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

role: front-end Front End Developer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Dev: Fix sass @import deprecation warnings by changing to @use

3 participants