Skip to content

Switched MetroReport to use the stack trace scrubber like the classic re...#142

Merged
JakeGinnivan merged 3 commits intoTestStack:masterfrom
JakeGinnivan:MetroReportUsingStackTraceScrubber
May 7, 2014
Merged

Switched MetroReport to use the stack trace scrubber like the classic re...#142
JakeGinnivan merged 3 commits intoTestStack:masterfrom
JakeGinnivan:MetroReportUsingStackTraceScrubber

Conversation

@JakeGinnivan
Copy link
Member

...port, fixes #105

@JakeGinnivan JakeGinnivan added this to the V4 milestone May 7, 2014
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

try-catch is formatted weirdly!

@MehdiK
Copy link
Member

MehdiK commented May 7, 2014

Thanks. Apart from the try appearing on a wrong line, it's cool.

Please fix that and it's good for merge.

@JakeGinnivan
Copy link
Member Author

k, fixed

JakeGinnivan added a commit that referenced this pull request May 7, 2014
…crubber

Switched MetroReport to use the stack trace scrubber like the classic re...
@JakeGinnivan JakeGinnivan merged commit cff0a51 into TestStack:master May 7, 2014
@JakeGinnivan JakeGinnivan deleted the MetroReportUsingStackTraceScrubber branch May 7, 2014 15:26
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.

Drop FakeExceptionWithStackTrace in Metro tests & use StackTraceScrubber

2 participants