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

chore(deps): Update dependency @cypress/react-tooltip to version 0.5.0 🌟 #4610

Merged
merged 4 commits into from Jul 3, 2019

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Jun 29, 2019

This PR contains the following updates:

Package Type Update Change
@cypress/react-tooltip devDependencies minor 0.4.0 -> 0.5.0

Release Notes

cypress-io/react-tooltip

v0.5.0

Compare Source

Bug Fixes
  • use latest semantic release (fab1d26)
Features

Renovate configuration

πŸ“… Schedule: "after 2am and before 4am" in timezone America/New_York.

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻️ Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".

πŸ”• Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot. View repository job log here.

@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Jul 1, 2019

Hey @chrisbreiding, there's some failures in our component tests surrounding the tooltips when we upgrade to the most recent cypress-io/react-tooltip version. Can you take a look?

@renovate
Copy link
Contributor Author

@renovate renovate bot commented Jul 1, 2019

PR has been edited

πŸ‘· This PR has received other commits, so Renovate will stop updating it to avoid conflicts or other problems. If you wish to abandon your changes and have Renovate start over you may click the "rebase" checkbox in the PR body/description.

@jennifer-shehane
Copy link
Member

@jennifer-shehane jennifer-shehane commented Jul 2, 2019

@chrisbreiding πŸ˜… is this done? I don't understand this random El code exactly. πŸ€”

@chrisbreiding
Copy link
Contributor

@chrisbreiding chrisbreiding commented Jul 2, 2019

Yes, it's done. The tests were passing in something element-like before, but with the new version, it looks like the contains function is stricter and requires an actual element.

@jennifer-shehane jennifer-shehane self-requested a review Jul 3, 2019
@jennifer-shehane jennifer-shehane merged commit bfcfe83 into develop Jul 3, 2019
23 checks passed
@renovate renovate bot deleted the renovate/@cypress branch Jul 3, 2019
@cypress-bot
Copy link

@cypress-bot cypress-bot bot commented Jul 9, 2019

Released in 3.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants