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 peerDependency ranges more strict #344

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Methuselah96
Copy link
Contributor

@Methuselah96 Methuselah96 commented Apr 8, 2020

Based on the conversation here: #343. I'm not necessarily advocating for this change, but I created the PR so that we can talk about it separately from #343.

Copy link
Member

@ljharb ljharb left a comment

Choose a reason for hiding this comment

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

LGTM, noting that tightening peer deps is a breaking change.

packages/jest-environment-enzyme/package.json Outdated Show resolved Hide resolved
packages/jest-enzyme/package.json Outdated Show resolved Hide resolved
Methuselah96 and others added 3 commits April 8, 2020 13:25
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
Co-Authored-By: Jordan Harband <ljharb@gmail.com>
@danez
Copy link

danez commented Dec 12, 2020

Can we add jest 26 and cut a release please?

@GreenGremlin
Copy link
Contributor

Yes! Please add jest v26! It's very discouraging that it is taking so long to support newer versions of jest, especially when it's such a simple fix.

"cpy-cli": "^2.0.0",
"enzyme": "^3.4.0",
"jest": "^22.0.0 || ^23.0.0 || ^24.0.0 || ^25.0.0",
"react": "^0.13.0 || ^0.14.0 || ^15.0.0 || ^16.0.0"
},
"dependencies": {
"jest-environment-jsdom": "^25.2.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

The jest-environment-jsdom dependency needs to be just as wide as the jest dependency.

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.

None yet

4 participants