Skip to content

Support for io.js - #2

Merged
Calamari merged 3 commits into
errwischt:masterfrom
ide:iojs
Jul 28, 2015
Merged

Support for io.js#2
Calamari merged 3 commits into
errwischt:masterfrom
ide:iojs

Conversation

@ide

@ideide commented Jun 24, 2015

Copy link
Copy Markdown
Contributor
  • Added test case for a stack trace from io.js 2.3.0 (worked fine, didn't require changes)
  • Added Travis configuration

@ide

ide commented Jun 24, 2015

Copy link
Copy Markdown
ContributorAuthor

@Calamari are you interested in maintaining this repo? Wondering since there wasn't any feedback left on #1.

ide added 2 commits June 30, 2015 03:59
The pattern for Gecko-style traces can match both URLs that start with a protocol and also local file paths when running scripts in JavaScriptCore. Allow files to start with any non-space character which covers periods, hyphens, and underscores for example.
@ideide changed the title Support for JSC and io.jsSupport for io.jsJun 30, 2015
@ide

ide commented Jul 3, 2015

Copy link
Copy Markdown
ContributorAuthor

@Calamari sorry to bother you -- could you take a look at this PR?

@ide

ide commented Jul 16, 2015

Copy link
Copy Markdown
ContributorAuthor

@Calamari - ping

@Calamari

Copy link
Copy Markdown
Contributor

Pong.. I am currently without notebook on vacation since bit over 2 weeks
and for one more week. But I will take a look directly afterwards.
On 16 Jul 2015 02:57, "James Ide" notifications@github.com wrote:

@Calamarihttps://github.com/Calamari - ping


Reply to this email directly or view it on GitHub
#2 (comment)
.

@ide

ide commented Jul 18, 2015

Copy link
Copy Markdown
ContributorAuthor

Cool. Thanks for the update =)

Run on Node 0.10 and 0.12 and io.js 1.x and 2.x.
Calamari added a commit that referenced this pull request Jul 28, 2015
Support and test cases for io.js
@Calamari
Calamari merged commit bcff42f into errwischt:masterJul 28, 2015
@ide

ide commented Jul 28, 2015

Copy link
Copy Markdown
ContributorAuthor

Thanks! Could you publish a new release to npm?

Sign up for freeto 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.

2 participants

@ide@Calamari