Skip to content

Conversation

@przemyslawzalewski
Copy link
Member

Closes #104, closes #106, closes #107, closes #108, closes #109, closes #110, closes #111, closes #112, closes #113, closes #114, closes #115, closes #116, closes #117 and closes #118.

marekrozmus
marekrozmus previously approved these changes Mar 12, 2020
Copy link

@marekrozmus marekrozmus left a comment

Choose a reason for hiding this comment

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

@przemyslawzalewski Should we bump up peer dependencies?

@przemyslawzalewski
Copy link
Member Author

@przemyslawzalewski Should we bump up peer dependencies?

I've thought you were sleeping 😆 You're right, if we don't have to, it would be better to keep them as they are.

@codecov-io
Copy link

codecov-io commented Mar 12, 2020

Codecov Report

Merging #121 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #121   +/-   ##
=======================================
  Coverage   66.49%   66.49%           
=======================================
  Files           2        2           
  Lines         191      191           
  Branches       51       51           
=======================================
  Hits          127      127           
  Misses         22       22           
  Partials       42       42

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 8833402...334ec3c. Read the comment docs.

@marekrozmus marekrozmus merged commit 153826b into master Mar 12, 2020
@marekrozmus marekrozmus deleted the update_dependencies_11_03_2020 branch March 12, 2020 07:50
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