Skip to content

Remove deprecation warnings in the build logs - #866

Merged
Dananji merged 1 commit into
mainfrom
build-warnings
Sep 2, 2025
Merged

Remove deprecation warnings in the build logs#866
Dananji merged 1 commit into
mainfrom
build-warnings

Conversation

@Dananji

@Dananji Dananji commented Aug 29, 2025

Copy link
Copy Markdown
Collaborator

This PR fixes the following deprecation warnings in the build logs;

Issue:

Deprecation Warning [import]: Sass @import rules are deprecated and will be removed in Dart Sass 3.0.0.

More info and automated migrator: https://sass-lang.com/d/import

  ╷
1 │ @import '../../styles/vars';
  │         ^^^^^^^^^^^^^^^^^^^

Fix from https://sass-lang.com/documentation/breaking-changes/import/:

npm install -g sass-migrator
sass-migrator module --migrate-deps <filename>.scss

Issue:

Deprecation Warning [legacy-js-api]: The legacy JS API is deprecated and will be removed in Dart Sass 2.0.0.

More info: https://sass-lang.com/d/legacy-js-api

Fix: Upgrade sass-loader used in Webpack server and Styleguidist docs configurations. Even though the main build doesn't need these configurations, they are run along with the build command, thus issuing the deprecation warnings.

Related issues: #837, #862

@cjcolvar cjcolvar left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good!

@Dananji
Dananji merged commit 09f3e95 into main Sep 2, 2025
2 checks passed
@Dananji
Dananji deleted the build-warnings branch September 2, 2025 20:55
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.

2 participants