Skip to content

Remove the hdf dep#992

Merged
adam-urbanczyk merged 1 commit intomasterfrom
hdf-dep-check
Feb 13, 2022
Merged

Remove the hdf dep#992
adam-urbanczyk merged 1 commit intomasterfrom
hdf-dep-check

Conversation

@adam-urbanczyk
Copy link
Copy Markdown
Member

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

Remove the pinned hdf dep

@codecov
Copy link
Copy Markdown

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
Copy Markdown
Member Author

Trivial change - merging

@whophil
Copy link
Copy Markdown

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.

@jmwright jmwright deleted the hdf-dep-check branch October 28, 2025 20:16
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