Skip to content

Conversation

@SimenB
Copy link
Member

@SimenB SimenB commented Jul 15, 2019

Summary

These should all still support node 6

Test plan

Green CI

@SimenB
Copy link
Member Author

SimenB commented Jul 15, 2019

I'm not sure why a couple of the integration tests fail...

@SimenB SimenB mentioned this pull request Jul 15, 2019
@SimenB
Copy link
Member Author

SimenB commented Jul 15, 2019

Trying #8693, then I can debug this afterwards. I suspect something with the babel deps (I kept some back due to babel/babel#9621, might be some cross-version errors)

@SimenB SimenB force-pushed the refresh-lockfile branch from 2dfe955 to d795620 Compare July 16, 2019 09:04
@SimenB SimenB force-pushed the refresh-lockfile branch from d795620 to af3e109 Compare July 16, 2019 11:10
@SimenB SimenB force-pushed the refresh-lockfile branch from af3e109 to d4debe4 Compare July 16, 2019 11:10
"@types/graceful-fs": "^4.1.2",
"@types/micromatch": "^3.1.0",
"@types/p-each-series": "^1.0.0",
"@types/p-each-series": "1.0.0",
Copy link
Member Author

@SimenB SimenB Jul 16, 2019

Choose a reason for hiding this comment

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

newer versions of p-each-series has types, but they are node 8 only. for some reason the types published as 1.0.1 makes type check fail for us...

) {
const ERROR_REGEX = /^Error:?\s\n*/;

if (stack && message && !stack.includes(message)) {
Copy link
Member Author

Choose a reason for hiding this comment

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

it doesn't hurt to always try to replace

// remove the prefix from the message because it is generally not useful.
const messageMatch = content.match(
/^(?:Error: )?([\s\S]*?(?=\n\s*at\s.*\:\d*\:\d*)|\s*.*)([\s\S]*)$/,
/^(?:Error: )?([\s\S]*?(?=\n\s*at\s.*:\d*:\d*)|\s*.*)([\s\S]*)$/,
Copy link
Member Author

Choose a reason for hiding this comment

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

intellij told me I didn't need to escape the :s

@SimenB SimenB merged commit 4a7b28c into jestjs:master Jul 16, 2019
@SimenB SimenB deleted the refresh-lockfile branch July 16, 2019 14:31
@github-actions
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants