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

Bump react-onclickoutside to 6.12.1 #3653

Merged

Conversation

patrykboch
Copy link
Contributor

@patrykboch patrykboch commented Jun 2, 2022

- the `react-onclickoutside` in v. 6.12.1 specifies React 18 in`peerDependencies`
- as a result annoying incompatibility warning flies away
@patrykboch
Copy link
Contributor Author

@martijnrusschen please review

@codecov
Copy link

codecov bot commented Jun 6, 2022

Codecov Report

Merging #3653 (7a1e253) into master (4441795) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3653   +/-   ##
=======================================
  Coverage   93.34%   93.34%           
=======================================
  Files          20       20           
  Lines        1352     1352           
  Branches      377      377           
=======================================
  Hits         1262     1262           
  Misses         27       27           
  Partials       63       63           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4441795...7a1e253. Read the comment docs.

@martijnrusschen martijnrusschen merged commit f4a6cfe into Hacker0x01:master Jun 6, 2022
@patrykboch patrykboch deleted the bump-react-onlickoutside branch June 6, 2022 08:51
p-rogas pushed a commit to p-rogas/react-datepicker that referenced this pull request Mar 22, 2023
…utside

Bump `react-onclickoutside` to 6.12.1
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

2 participants