-
Notifications
You must be signed in to change notification settings - Fork 465
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
[WIP] Early preview of dart-sass parser backport #2918
Conversation
3172461
to
95bedc6
Compare
Current state: 26 specs with (mostly minor) regressions, 122 newly passing |
3923e61
to
6835c29
Compare
Current state: all specs passing with around 150 newly passing Ported scss, sass and css parser.
Master: 3062 runs, 0 assertions, 0 failures, 0 errors, 829 skips |
98e8d55
to
4613f3e
Compare
Current state: 3906 runs, 0 assertions, 0 failures, 0 errors, 752 skips |
What do you see as the next step to getting this landed?
…On Sat., 20 Jul. 2019, 12:50 pm Marcel Greter, ***@***.***> wrote:
Current state: 3906 runs, 0 assertions, 0 failures, 0 errors, 752 skips
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#2918?email_source=notifications&email_token=AAENSWFWBI5SYVFQZXVA2FDQAJ4PBA5CNFSM4H2VW2A2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2NE6MI#issuecomment-513429297>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAENSWA2KXYFYRTF5EQYKULQAJ4PBANCNFSM4H2VW2AQ>
.
|
34059f0
to
abebb51
Compare
@xzyfer Don't expect anything to land before winter. |
5b17a56
to
3de51e9
Compare
46ee11c
to
5e345c0
Compare
3b38d57
to
993d5a7
Compare
45b1e0b
to
f1cfd65
Compare
57cddb1
to
b0ab13c
Compare
b0ab13c
to
32c6925
Compare
2fbe0e0
to
1c35cbc
Compare
Current stats: 4143 runs, 0 assertions, 0 failures, 0 errors, 148 skips |
1c35cbc
to
96c4224
Compare
96c4224
to
6ad8048
Compare
Current stats: 5814 runs, 0 assertions, 0 failures, 0 errors, 1196 skips |
Took long enough, but will now try to go for a LibSass 4.0 alpha release. @xzyfer I hope no objections from you to move this forward? |
🚢 agree the deprecation means there are no expectations on the state of master. |
Continued in #3135 |
Needs sass/sass-spec#1425
Please don't review, only leaving this here since I don't know when I'll have time to finish this up.
My vacations are over soon, so I don't know when I'll get around to complete this.
It's already semi usable, passing 80% of current spec tests.
Tried to preserve current libsass behavior, but
probably needs changes in eval stage too!
All the debug glory is still in the code!
Edit: I started to bring back some stuff to master in order to make this change a little bit smaller.
I also tagged all open issues that should be fixed by this backport/refactoring (ATM 76/120).