Skip to content

Import the TypeScript definition for StackFrame from stackframe - #228

Open
andersk wants to merge 1 commit into
stacktracejs:masterfrom
andersk:typescript-import
Open

Import the TypeScript definition for StackFrame from stackframe#228
andersk wants to merge 1 commit into
stacktracejs:masterfrom
andersk:typescript-import

Conversation

@andersk

Copy link
Copy Markdown
Contributor

Description

The slight incompatibility of the old definition was necessitating unsafe casts between StackTrace.StackFrame and StackFrame. See also stacktracejs/error-stack-parser#54.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • node_modules/.bin/jscs -c .jscsrc stacktrace.js passes without errors
    • You don’t use JSCS anymore.
  • npm test passes without errors
  • I have read the contribution guidelines
  • I have updated the documentation accordingly
  • I have added tests to cover my changes

The slight incompatibility of the old definition was necessitating
unsafe casts between StackTrace.StackFrame and StackFrame.
Signed-off-by: Anders Kaseorg <andersk@mit.edu>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@andersk