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

NODESASS_TRACE: Live tracing facility#1007

Open
saper wants to merge 2 commits into
sass:masterfrom
saper:logging
Open

NODESASS_TRACE: Live tracing facility#1007
saper wants to merge 2 commits into
sass:masterfrom
saper:logging

Conversation

@saper

@saper saper commented Jun 21, 2015

Copy link
Copy Markdown
Member

Introduce the ultimate libsass C API
troubleshooting tool.

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

@saper

saper commented Jun 23, 2015

Copy link
Copy Markdown
Member Author

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

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

@saper

saper commented Aug 25, 2015

Copy link
Copy Markdown
Member Author

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

@xzyfer

xzyfer commented Aug 26, 2015

Copy link
Copy Markdown
Contributor

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

@xzyfer

xzyfer commented Aug 26, 2015

Copy link
Copy Markdown
Contributor

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

xzyfer commented Aug 31, 2015

Copy link
Copy Markdown
Contributor

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

@saper

saper commented Aug 31, 2015

Copy link
Copy Markdown
Member Author

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

xzyfer commented Sep 1, 2015

Copy link
Copy Markdown
Contributor

👍 for the when we land libsass@3.3

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

xzyfer commented Oct 24, 2015

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Contributor

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

@xzyfer

xzyfer commented Apr 30, 2016

Copy link
Copy Markdown
Contributor

@saper is this something you still want?

@saper

saper commented Apr 30, 2016 via email

Copy link
Copy Markdown
Member Author

@xzyfer

xzyfer commented Apr 30, 2016

Copy link
Copy Markdown
Contributor

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

xzyfer commented Sep 8, 2016

Copy link
Copy Markdown
Contributor

@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