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

Make @jest/reporters a peerDependency, and bump the required version down #1

Merged
merged 4 commits into from
Jan 18, 2021

Conversation

sophiekoonin
Copy link
Contributor

This way, it won't install files from a newer major version of Jest in repos that are still on v25.

The tool runs perfectly well in v25, so figured it may as well require that rather than v26.

@jevakallio
Copy link
Owner

Thanks @sophiekoonin, looks great! I'll test it this evening and release a new version once I've verified it doesn't cause any havoc in my own project.

"chalk": "^4.1.0"
},
"peerDependencies": {
"@jest/reporters": "^25.0.0"
Copy link

Choose a reason for hiding this comment

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

Suggested change
"@jest/reporters": "^25.0.0"
"@jest/reporters": "^25.1.0 || ^26.0.0"

25.1.0 was first stable version of 25 (a version that was supposed to be a pre-release may or may not have been tagged as latest. npm is hard sometimes 👼).

Also, should still support v26

@jevakallio jevakallio merged commit 56a00a0 into jevakallio:master Jan 18, 2021
@jevakallio
Copy link
Owner

jevakallio commented Jan 18, 2021

Released as 0.2.0.

Note that in order to stop npm complaing, I took on a peer dependency to jest instead of the individual @jest packages.

Thanks @sophiekoonin! 🤝

Thanks @SimenB for reviewing!

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.

3 participants