Skip to content

fix(scss): migrate @import to @use, address npm vulnerabilities#406

Merged
vytas7 merged 2 commits intomasterfrom
scss-warnings
Mar 29, 2026
Merged

fix(scss): migrate @import to @use, address npm vulnerabilities#406
vytas7 merged 2 commits intomasterfrom
scss-warnings

Conversation

@vytas7
Copy link
Copy Markdown
Member

@vytas7 vytas7 commented Mar 29, 2026

Replace deprecated Sass @import rules in theme.scss with @use, passing Bootstrap variable overrides via the with() configuration block.

Run npm audit fix to resolve 3 vulnerabilities (brace-expansion, path-to-regexp, serialize-javascript).

Disclaimer: this change was partially automated with Claude Code. 🤖

Fixes #320.

Replace deprecated Sass `@import` rules in `theme.scss` with `@use`, passing
Bootstrap variable overrides via the `with()` configuration block.

Run npm audit fix to resolve 3 vulnerabilities
(brace-expansion, path-to-regexp, serialize-javascript).
@vytas7 vytas7 requested a review from CaselIT March 29, 2026 17:52
Copy link
Copy Markdown
Member

@CaselIT CaselIT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know much about this, but that change looks fine.

Were all the variables also updated before?

@vytas7
Copy link
Copy Markdown
Member Author

vytas7 commented Mar 29, 2026

Not sure about the variables, as I understand they were inlined only now. I have verified the look is unchanged.

@vytas7 vytas7 merged commit 45d8d07 into master Mar 29, 2026
2 checks passed
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.

Address SCSS warnings

2 participants