search

mhe / pynrrd

Simple pure-python module for reading and writing nrrd files.
https://pynrrd.readthedocs.io/
MIT License
116 stars 51 forks source link

Update the documentation with ALLOW_DUPLICATE_FIELD #72

Closed jcnils closed 6 years ago

jcnils commented 6 years ago

Documentation update to include the ALLOW_DUPLICATE_FIELD information (#71 )

I am not used to Sphinx, but I update it using the sphinx-doc by hand. http://www.sphinx-doc.org/en/master/usage/restructuredtext/domains.html

I hope it is close to the expected.

codecov-io commented 6 years ago

Codecov Report

Merging #72 into master will not change coverage. The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #72   +/-   ##
=======================================
  Coverage   87.39%   87.39%           
=======================================
  Files           6        6           
  Lines         357      357           
  Branches      114      114           
=======================================
  Hits          312      312           
  Misses         22       22           
  Partials       23       23
Impacted Files Coverage Δ
nrrd/reader.py 81.98% <100%> (ø) :arrow_up:

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 f3097a5...512b451. Read the comment docs.

addisonElliott commented 6 years ago

I mentioned some of this in my commit, but I pushed some changes to your branch (hope that's okay). I had to get your changes to compile them and check out the docs, so I figured I might as well change the documentation while I had it.

Once I get your approval on this, I will merge it.

jcnils commented 6 years ago

Those are perfect, thank you very much @addisonElliott