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

Added source-map-support library which helps with stack traces. #16

Merged
merged 1 commit into from
Feb 23, 2016

Conversation

jmurphyau
Copy link
Contributor

This library needs some assistance to attach to Error.prepareStackTrace outside the VM then again inside the VM. Internally it keeps track of a errorFormatterInstalled flag - this flag stops any future calls to install on Error.prepareStackTrace after the first one, which is usually the one outside the VM.

This library needs some assistance to attach to `Error.prepareStackTrace` outside the VM then again inside the VM. Internally it keeps track of a `errorFormatterInstalled` flag - this flag stops any future calls to install on `Error.prepareStackTrace` after the first one, which is usually the one outside the VM.
@jmurphyau
Copy link
Contributor Author

Before:
1-before

After:

1-after

Before:
2-before

After:

2-after

Before:
3-before

After:
3-after

@rwjblue
Copy link
Member

rwjblue commented Feb 12, 2016

Looks awesome!

@jmurphyau
Copy link
Contributor Author

@tomdale have you had a chance to look into this one?

@tomdale
Copy link
Contributor

tomdale commented Feb 23, 2016

@jmurphyau Hey! I just saw it today, unfortunately—somehow I was not watching this repo and not getting any emails. I'll take a look. Sorry for the delay!

tomdale added a commit that referenced this pull request Feb 23, 2016
Added source-map-support library which helps with stack traces.
@tomdale tomdale merged commit 6f88523 into ember-fastboot:master Feb 23, 2016
bantic pushed a commit to bantic/ember-fastboot-server that referenced this pull request Apr 28, 2016
Use separate output path for `ember fastboot`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants