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.

Sign up for GitHub

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

Jump to bottom

Update stackframe to 1.x.x #28

Closed
wants to merge 22 commits into from
Closed

Update stackframe to 1.x.x #28

wants to merge 22 commits into from

Conversation

btd
Copy link

@btd btd commented Jan 5, 2016

No description provided.

ssenbud and others added 22 commits September 20, 2015 17:51
@ssenbud
Update stackframe dependency.
1d24701
@oliversalzburg
Merge branch 'master' of github.com:stacktracejs/error-stack-parser
b058de2
@oliversalzburg
Full semver for bower dependency
ce53af0
@oliversalzburg
Bower cleanup
883bce1
@ssenbud
Enable Sauce for all supported browsers
46e2f7c
@ssenbud
Avoid premature/lossy normalization of function names in IE/Chrome. F…
ec2f79d 
…ixes stacktracejs#17 and stacktracejs#19
@ssenbud
Specify Windows for Firefox tests in the hopes it'll work reliably on…
6d1b89b 
… Sauce.
@ssenbud
Remove Firefox beta from Sauce testing because it keeps failing due t…
e1bb634 
…o unavailablity.
@ssenbud
Deduplicate browser support sections
1d90870
@ssenbud
Exclude eval information when parsing Chrome stacks. Fixes stacktrace…
d6dc380 
…js#25
@ssenbud
Improve handling of eval() stack frames.
53dd6f8 
This removes parts of stack lines related to the eval lines and columns.
This was done this way to avoid breaking current stack traces until we
have a good solution for including them.
@ssenbud
Switch to Jasmine 2 for testing.
7bcdfd1 
Motivation:
Focused specs, better ongoing support

Modifications:
Added jasmine and jasmine-core to package.json. Updated other
dependencies.
@ssenbud
Remove IE7 from CI - too unreliable.
785d79b
@ssenbud
Allow stacktraces with no location information in Safari. Should fix s…
f5c0c80 
…tacktracejs#16.
@ssenbud
Release 1.3.2
3e56eae
@ssenbud
Remove .nvmrc
bc9d544
@ssenbud
Add test task for PRs
55e682b
@ssenbud
Don't use Sauce for PRs, use local browsers.
0e572cc
@ssenbud
Replace use of Array.map and Array.filter in order to avoid the need …
b6d8241 
…for polyfills. Release v1.3.3
@leonidborisenko
Fully parse function names with @ in Error.stack
b783149 
Function name in Error.stack can contain @ when function is defined as an
object property:

    const obj = { '@fn'() { throw new Error(); } };
    obj['@fn']();
    /*
    Exception: Error
    obj["@fn"]@Scratchpad/1:10:31
    @Scratchpad/1:11:1
    */
@eriwen
Merge pull request stacktracejs#27 from leonidborisenko/fully-parse-f…
e3eae6a 
…n-name-with-at-sign

Fully parse function names with @ in Error.stack
@btd
Update stackframe to 1.x.x
5f850c9
@btd
Copy link
Author

btd commented Jan 8, 2016

Is anything there stopping from merge? Locally all tests passed.

@eriwen
Copy link
Member

eriwen commented Jan 8, 2016

Sorry for the delay here. This change may break downstream libraries because they cannot construct StackFrames the same way. It may be better to keep this off of master just until we're ready to implement parsing eval origins etc. for v2.0.

Do you have any urgent need for this change?

@btd
Copy link
Author

btd commented Jan 8, 2016

Not really urgent need, just to do not have in client bundle them twice.

@btd btd closed this Sep 26, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@btd @eriwen @ssenbud @oliversalzburg @leonidborisenko