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.

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

Upgrade jest-junit to 6.2.0 #7739

Merged
merged 3 commits into from
Jan 28, 2019
Merged

Upgrade jest-junit to 6.2.0 #7739

merged 3 commits into from
Jan 28, 2019

Conversation

palmerj3
Copy link
Contributor

Summary

Updates jest-junit to ^6.2.0.

jest-junit 6.2.0 removes the dependency on jest-config so it removes babel-core and babel-jest so it should reduce the install time and be more compatible with future releases.

Test plan

This PR

@palmerj3
Copy link
Contributor Author

This uncovered an issue with jest-junit where node 6 can't handle trailing commas for function params. Pushing a fix to jest-junit and will update this PR.

Copy link
Contributor

@jeysal jeysal left a comment

Choose a reason for hiding this comment

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

AppVeyor failure looks unrelated, but I haven't seen it fail that test before. Rerun?

@palmerj3
Copy link
Contributor Author

What's the easiest way to retrigger?

@@ -2182,31 +2182,6 @@ [email protected], babel-code-frame@^6.26.0:
esutils "^2.0.2"
js-tokens "^3.0.2"

babel-core@^6.0.0, babel-core@^6.26.0:
Copy link
Member

Choose a reason for hiding this comment

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

yay, babel-core gona from our dep tree 😀

@SimenB SimenB merged commit 2e2d2c8 into jestjs:master Jan 28, 2019
@SimenB
Copy link
Member

SimenB commented Jan 28, 2019

What's the easiest way to retrigger?

Unrelated (we only use jest-junit on circle)

@palmerj3
Copy link
Contributor Author

Woo!

@palmerj3 palmerj3 deleted the jestJunit branch January 28, 2019 22:28
captain-yossarian pushed a commit to captain-yossarian/jest that referenced this pull request Jul 18, 2019
@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 12, 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.

5 participants