Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

NODESASS_TRACE: Live tracing facility #1007

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

saper
Copy link
Member

@saper saper commented Jun 21, 2015

Introduce the ultimate libsass C API
troubleshooting tool.

See your JavaScript values being folded
into the libsass context. And back.

@saper
Copy link
Member Author

saper commented Jun 23, 2015

Same output (with sass/libsass#1288 enabled as well):

https://gist.github.com/saper/43dbc3c050553d035692

@saper
Copy link
Member Author

saper commented Aug 25, 2015

ping! it merges with the current master :) 7c15014

@xzyfer
Copy link
Contributor

xzyfer commented Aug 26, 2015

This would mean rebuilding the 3.3.0 binaries, so not this time :)

@xzyfer
Copy link
Contributor

xzyfer commented Aug 26, 2015

I'd also like to seen some docs how to use this added. I'd like to play with it before shipping it.

@saper saper added this to the next.patch milestone Aug 29, 2015
@xzyfer xzyfer modified the milestones: next.minor, next.patch Aug 31, 2015
@xzyfer
Copy link
Contributor

xzyfer commented Aug 31, 2015

I assume this needs sass/libsass#1288 to of any use @saper?

@saper
Copy link
Member Author

saper commented Aug 31, 2015

Nope, it has the same implementation though (could be shared maybe). Will work on this, since it has proven already useful e.g. to triage out libsass bugs.

@xzyfer
Copy link
Contributor

xzyfer commented Sep 1, 2015

👍 for the when we land [email protected]

@xzyfer xzyfer modified the milestones: next.libsass, next.minor Sep 1, 2015
@xzyfer
Copy link
Contributor

xzyfer commented Oct 24, 2015

Moving this to next.patch milestone. Let's get this sorted.

@xzyfer xzyfer modified the milestones: next.patch, next.libsass Oct 24, 2015
@saper saper force-pushed the master branch 2 times, most recently from 6c128d9 to 1e4bba8 Compare April 19, 2016 21:45
@nschonni
Copy link
Contributor

@xzyfer are you going to manually merge this for 3.7, or bump to the next one?

@xzyfer
Copy link
Contributor

xzyfer commented Apr 30, 2016

@saper is this something you still want?

@saper
Copy link
Member Author

saper commented Apr 30, 2016 via email

@xzyfer
Copy link
Contributor

xzyfer commented Apr 30, 2016

Rebase and :shipit:

Set NODESASS_TRACE enviroment variable
to any value to get lots of output.
Introduce one true way to print contents
of the SassValue union.
@xzyfer
Copy link
Contributor

xzyfer commented Sep 8, 2016

@saper feel free to merge if you're still interested in this :)

@nschonni nschonni modified the milestones: 3.11.0, 3.10.0 Sep 15, 2016
@nschonni nschonni modified the milestones: 3.11.0, 3.12.0 Nov 8, 2016
@nschonni nschonni modified the milestones: 3.12.0, 3.14.0 Dec 5, 2016
@xzyfer xzyfer removed this from the 3.14.0 milestone Jan 12, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants