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

Document that header object given to nrrd.write will be updated with auto generated fields #86

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

addisonElliott
Copy link
Collaborator

Includes some minor fixes to the docs that were causing problems.

…ith the auto generated fields.

Also, fixed some minor issues with the docs.
@addisonElliott
Copy link
Collaborator Author

@simeks Mind reviewing this when you get a chance?

I'm going to go ahead and merge yours now.

@codecov-io
Copy link

codecov-io commented Mar 12, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #86   +/-   ##
=======================================
  Coverage   99.16%   99.16%           
=======================================
  Files           6        6           
  Lines         361      361           
  Branches      116      116           
=======================================
  Hits          358      358           
  Misses          1        1           
  Partials        2        2
Impacted Files Coverage Δ
nrrd/writer.py 97.43% <ø> (ø) ⬆️

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 07d2227...478199a. Read the comment docs.

@simeks
Copy link
Contributor

simeks commented Mar 13, 2019

@simeks Mind reviewing this when you get a chance?

Looks good! Makes more sense now when you actually see it in the docs 👍.

@addisonElliott addisonElliott merged commit 0dfddeb into master Mar 13, 2019
@addisonElliott addisonElliott deleted the update-docs-mutable-header branch March 13, 2019 12: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.

3 participants