Skip to content
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

Closed
wants to merge 2 commits into from

Conversation

mgreter
Copy link
Contributor

@mgreter mgreter commented Jun 22, 2019

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).

@mgreter
Copy link
Contributor Author

mgreter commented Jul 9, 2019

Current state: 26 specs with (mostly minor) regressions, 122 newly passing

@mgreter mgreter force-pushed the feature/backport-parser branch 5 times, most recently from 3923e61 to 6835c29 Compare July 15, 2019 01:16
@mgreter
Copy link
Contributor Author

mgreter commented Jul 15, 2019

Current state: all specs passing with around 150 newly passing

Ported scss, sass and css parser.
Lot of stuff still to do; most obvious:

  • ParserStates for sourcemaps
  • And correct error reporting
  • Cleanup of import classes

Master: 3062 runs, 0 assertions, 0 failures, 0 errors, 829 skips
Branch: 3725 runs, 0 assertions, 0 failures, 0 errors, 748 skips

@mgreter mgreter force-pushed the feature/backport-parser branch 7 times, most recently from 98e8d55 to 4613f3e Compare July 19, 2019 03:51
@mgreter
Copy link
Contributor Author

mgreter commented Jul 20, 2019

Current state: 3906 runs, 0 assertions, 0 failures, 0 errors, 752 skips

@xzyfer
Copy link
Contributor

xzyfer commented Jul 20, 2019 via email

@mgreter mgreter force-pushed the feature/backport-parser branch from 34059f0 to abebb51 Compare July 20, 2019 17:34
@mgreter
Copy link
Contributor Author

mgreter commented Jul 21, 2019

@xzyfer Don't expect anything to land before winter.
I guess this should land for a 3.8 version (close to sass 4.0).

@mgreter mgreter force-pushed the feature/backport-parser branch from 5b17a56 to 3de51e9 Compare July 21, 2019 05:18
@mgreter mgreter added the LibSass 3.8 Still not sass 4.0 label Jul 21, 2019
@mgreter mgreter self-assigned this Jul 21, 2019
@mgreter mgreter force-pushed the feature/backport-parser branch 5 times, most recently from 46ee11c to 5e345c0 Compare September 17, 2019 20:33
@mgreter mgreter force-pushed the feature/backport-parser branch from 3b38d57 to 993d5a7 Compare October 2, 2019 21:42
@mgreter mgreter added LibSass 4.0 Close to sass 4.0 and removed LibSass 3.8 Still not sass 4.0 labels Nov 8, 2019
@mgreter mgreter force-pushed the feature/backport-parser branch 2 times, most recently from 45b1e0b to f1cfd65 Compare November 8, 2019 04:29
@mgreter mgreter changed the title [WIP] Ultra early preview of dart-sass parser backport [WIP] Early preview of dart-sass parser backport Nov 15, 2019
@mgreter mgreter force-pushed the feature/backport-parser branch from 57cddb1 to b0ab13c Compare November 15, 2019 00:45
@mgreter mgreter force-pushed the feature/backport-parser branch from b0ab13c to 32c6925 Compare November 24, 2019 03:36
@mgreter mgreter force-pushed the feature/backport-parser branch 4 times, most recently from 2fbe0e0 to 1c35cbc Compare December 14, 2019 17:20
@mgreter
Copy link
Contributor Author

mgreter commented Dec 14, 2019

Current stats: 4143 runs, 0 assertions, 0 failures, 0 errors, 148 skips

@mgreter mgreter force-pushed the feature/backport-parser branch from 96c4224 to 6ad8048 Compare November 6, 2020 00:59
@mgreter
Copy link
Contributor Author

mgreter commented Nov 6, 2020

Current stats: 5814 runs, 0 assertions, 0 failures, 0 errors, 1196 skips
Lots of new spec tests, mostly failing regarding @use, @forward and new math module.

@mgreter
Copy link
Contributor Author

mgreter commented Nov 6, 2020

Took long enough, but will now try to go for a LibSass 4.0 alpha release.
Still want to discuss the C-API, it already has been simplified internally.
Once I get CI stable here I will open a new PR for getting that ready.
I hope to have a 4.0 beta out by start to middle 2021.

@xzyfer I hope no objections from you to move this forward?
Since I pretty much rewrote everything, a PR review seems impossible!
But I guess since we're officially deprecated, we have more leeway!?
Alternatively I could publish everything within my own forks?

@xzyfer
Copy link
Contributor

xzyfer commented Nov 6, 2020

🚢 agree the deprecation means there are no expectations on the state of master.

@mgreter mgreter mentioned this pull request Nov 6, 2020
19 tasks
@mgreter
Copy link
Contributor Author

mgreter commented Nov 6, 2020

Continued in #3135

@mgreter mgreter closed this Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants