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

Remove the hdf dep #992

Merged
merged 1 commit into from
Feb 13, 2022
Merged

Remove the hdf dep #992

merged 1 commit into from
Feb 13, 2022

Conversation

adam-urbanczyk
Copy link
Member

@adam-urbanczyk adam-urbanczyk commented Feb 11, 2022

Remove the pinned hdf dep

@codecov
Copy link

codecov bot commented Feb 11, 2022

Codecov Report

Merging #992 (2b20eff) into master (7847224) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #992   +/-   ##
=======================================
  Coverage   96.17%   96.17%           
=======================================
  Files          40       40           
  Lines        9292     9292           
  Branches     1109     1109           
=======================================
  Hits         8937     8937           
  Misses        208      208           
  Partials      147      147           

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 7847224...2b20eff. Read the comment docs.

@adam-urbanczyk
Copy link
Member Author

Trivial change - merging

@whophil
Copy link

whophil commented Feb 26, 2022

Do you know what changed which allowed this pin to be removed? I have this same hdf5 pin in some other projects.

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

Successfully merging this pull request may close these issues.

2 participants