-
Notifications
You must be signed in to change notification settings - Fork 89
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Use precision 10 for specs by default. #1132
Conversation
@xzyfer It looks like changing the default precision is causing libsass to produce different colors for some reason. Can you take a look? |
Think the upstream issue is sass/libsass#2294 |
@mgreter Changing the precision that's used for Sass arithmetic shouldn't affect the colors from color functions, but it does. I think this is a LibSass bug, and it's blocking my ability to land this pull request without marking tons of specs as TODO for LibSass. Can you look into fixing that? |
@mgreter ping? |
LibSass doesn't seem to correctly handle numbers at the far end of precision. See sass/libsass#3788
Since there hasn't been any response from the LibSass folks on this, I guess I'll just mark those specs as TODO. |
Follow-up to sass#1132 I'm guessing these weren't caught because of warning todos (failures are ignore with probe-todo that runs on CI, even though it's just a warning todo).
Follow-up to #1132 I'm guessing these weren't caught because of warning todos (failures are ignore with probe-todo that runs on CI, even though it's just a warning todo).
See sass/sass#1122
[skip dart-sass]