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

fix: allow cypress@5 as peer dependency #149

Closed
wants to merge 1 commit into from
Closed

fix: allow cypress@5 as peer dependency #149

wants to merge 1 commit into from

Conversation

SimenB
Copy link
Contributor

@SimenB SimenB commented Aug 19, 2020

What:

Up peer dep to the fresh v5 release

Why:

To avoid a warning when installing

How:

GitHub UI

Checklist:

  • Documentation
  • Tests
  • Ready to be merged

Worked fine except for the warning locally 👍

Copy link

@lkappeler lkappeler left a comment

Choose a reason for hiding this comment

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

Run our integration tests locally with cypress v5 and the latest testing library version in two projects, worked all fine. (Besides the install warning)

@nickserv
Copy link
Member

What's the warning? Should we merge despite it or wait?

@SimenB
Copy link
Contributor Author

SimenB commented Aug 21, 2020

I'm assuming they meant that using v5 worked fine except for the peer dependency warning which this pr will fix

@lkappeler
Copy link

Exactly

@kentcdodds
Copy link
Member

Sorry, I didn't notice this PR and I updated it without triggering a release. I'll push one out now. Sorry I missed the PR.

@kentcdodds kentcdodds closed this Aug 21, 2020
kentcdodds added a commit that referenced this pull request Aug 21, 2020
@SimenB
Copy link
Contributor Author

SimenB commented Aug 21, 2020

No problem at all 👍

@SimenB SimenB deleted the patch-1 branch August 22, 2020 10:25
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.

4 participants