Skip to content
This repository was archived by the owner on Oct 24, 2025. It is now read-only.

Conversation

@mgreter
Copy link
Contributor

@mgreter mgreter commented Dec 15, 2014

This updates sassc to use the new C API recently introduced in libsass.
I hope it does not break in any way, but it is of course a major change!

@mgreter
Copy link
Contributor Author

mgreter commented Dec 15, 2014

Would be great if someone could review or even try this!
It's a pretty major change and I could have broken anything!
I believe it should work, but do you really trust me that much?
😄

@am11
Copy link
Contributor

am11 commented Dec 15, 2014

I trust your judgment (and mine solely based on TravisCI results)! 😃

@xzyfer xzyfer mentioned this pull request Dec 15, 2014
12 tasks
mgreter added a commit that referenced this pull request Dec 20, 2014
Use new functional libsass C API
@mgreter mgreter merged commit 072a70f into sass:master Dec 20, 2014
@mgreter
Copy link
Contributor Author

mgreter commented Dec 20, 2014

Nothing ventured, nothing gained ...

@mgreter mgreter deleted the features/new-c-api branch December 20, 2014 03:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants