Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

loichuder
Copy link
Contributor

Fix #1994

👶 steps for a first contribution. Let me know if I missed something in the process

@codecov
Copy link

codecov bot commented Jun 2, 2022

Codecov Report

Merging #2108 (8d760ff) into master (859788e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2108   +/-   ##
=======================================
  Coverage   90.00%   90.00%           
=======================================
  Files          17       17           
  Lines        2350     2350           
=======================================
  Hits         2115     2115           
  Misses        235      235           

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 859788e...8d760ff. Read the comment docs.

@takluyver takluyver added this to the 3.8 milestone Jun 13, 2022
@takluyver
Copy link
Member

Thanks, this looks good to me.

@takluyver takluyver merged commit 2719c6a into h5py:master Jun 13, 2022
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.

Documentation gives wrong default arguments for require_dataset
2 participants