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

Update to jsdom v12 #1274

Closed
wants to merge 4 commits into from
Closed

Conversation

sarvaje
Copy link
Contributor

@sarvaje sarvaje commented Aug 28, 2018

Pull request checklist

Make sure you:

For non-trivial changes, please make sure you also:

  • Added/Updated related documentation.
  • Added/Updated related tests.

Short description of the change(s)

@sarvaje sarvaje requested review from alrra and molant as code owners August 28, 2018 00:02
@sarvaje
Copy link
Contributor Author

sarvaje commented Aug 28, 2018

This pull request depends on jsdom/jsdom#2339 and jsdom/jsdom#2349. That's why right now the link to jsdom is to my repository.

alrra pushed a commit that referenced this pull request Sep 22, 2018
Depending on the package manager used for the installation,
different versions of `parse5` with different APIs could be
installed.

Until #1274 is merged, this solves that case.
alrra pushed a commit that referenced this pull request Sep 23, 2018
Depending on the package manager used for the installation,
different versions of `parse5` with different APIs could be
installed.

Until #1274 is merged, this solves that case.
alrra pushed a commit that referenced this pull request Sep 25, 2018
Depending on the package manager used for the installation, different
versions of `parse5` with different APIs could be installed.

Until #1274 is merged, this commit solves the issues specified above.
@alrra alrra force-pushed the master branch 2 times, most recently from e1993b0 to 6c3bfae Compare September 27, 2018 20:34
@molant molant mentioned this pull request Oct 1, 2018
4 tasks
@alrra
Copy link
Contributor

alrra commented Oct 1, 2018

Closing in favor of #1357.

@alrra alrra closed this Oct 1, 2018
@sarvaje sarvaje deleted the jsdom-update-v12 branch October 15, 2018 22:00
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