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

Add jest-environment-jsdom to peer deps #346

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Whoaa512
Copy link

@Whoaa512 Whoaa512 commented Jun 8, 2020

This is blocking a jest upgrade we're doing, since multiple copies of the jest-environment-jsdom is producing errors in our tests.

The other PR(#343) for this seems stalled out.

@ljharb
Copy link
Member

ljharb commented Jun 8, 2020

There's no need to submit a duplicate PR; #343 or #344 will be the one that's landed imo.

@Whoaa512 Whoaa512 closed this Nov 17, 2021
@Whoaa512 Whoaa512 deleted the jsdom-peer-dep branch November 17, 2021 09:32
@ljharb
Copy link
Member

ljharb commented Nov 17, 2021

@Whoaa512 would you please restore the branch? i need to keep both PRs open and in sync to avoid polluting the repo with the PR ref.

@Whoaa512 Whoaa512 restored the jsdom-peer-dep branch November 18, 2021 06:38
@Whoaa512
Copy link
Author

This is a duplicate as you've said. I'm trying to close out stale PRs that have seen no movement and this seems superseded by the 2 other PRs linked

@ljharb
Copy link
Member

ljharb commented Nov 18, 2021

Yes, but the PR ref will remain as a permanent mark on the repo, whereas if this PR can be left open, then I can update all three PRs to the same sha and land them all at once, later.

@ljharb ljharb reopened this May 17, 2023
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.

2 participants