include-media icon indicating copy to clipboard operation
include-media copied to clipboard

DEPRECATION WARNING: Using / for division is deprecated and will be removed in Dart Sass 2.0.0.

Open sstikkel opened this issue 3 years ago • 6 comments

I get a warning regarding line 443 $result: $result + map-get($numbers, $characters) / $digits; Recommendation: math.div(map-get($numbers, $character), $digits) and @use "sass:math":

sstikkel avatar Jul 14 '21 07:07 sstikkel

hey @sstikkel please can i see your file setup? I can't even manage to setup my own project. i will appreciate any help | #212

codiejay avatar Jul 14 '21 14:07 codiejay

I get a warning regarding line 443 $result: $result + map-get($numbers, $characters) / $digits; Recommendation: math.div(map-get($numbers, $character), $digits) and @use "sass:math":

This is because you are using a newer dart-sass version which encourages you to @use "sass:math" on top of the file and replace the division with math.div.

Supportic avatar Jul 14 '21 22:07 Supportic

Any updates on this? We should really fix this to be able to use the latest sass version without getting those division warnings.

alexanderalmstrom avatar Dec 17 '21 00:12 alexanderalmstrom

Yeah, might dedicate a day to this over the break. Thinking of a 2.0 release to fix the dart sass stuff and simplify issues.

jackmcpickle avatar Dec 17 '21 00:12 jackmcpickle

Note I've mainly switched to postcss, and i've written. https://www.npmjs.com/package/postcss-include-media

jackmcpickle avatar Dec 17 '21 00:12 jackmcpickle

I've created a pull request here https://github.com/eduardoboucas/include-media/pull/214

alexanderalmstrom avatar Dec 17 '21 00:12 alexanderalmstrom

resolved in 2.0 release https://github.com/eduardoboucas/include-media/pull/218

jackmcpickle avatar Jan 24 '23 11:01 jackmcpickle