Skip to content

1124: Silence SASS deprecation warnings (do not merge).#1125

Open
quicksketch wants to merge 1 commit intomainfrom
1124--silence-sass-warnings
Open

1124: Silence SASS deprecation warnings (do not merge).#1125
quicksketch wants to merge 1 commit intomainfrom
1124--silence-sass-warnings

Conversation

@quicksketch
Copy link
Copy Markdown
Member

Fixes #1124

This is a temporary hack to silence the deprecation warnings when running task sass. It is not the correct solution but it shows where the problem lies in the current SASS compilation warnings.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The added silenceDeprecations option is duplicated across two SASS compiler calls. These duplicated options should be extracted into a shared constant to improve maintainability.

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.

DEPRECATION WARNING: The legacy JS API is deprecated and will be removed in Dart Sass 2.0.0.

2 participants