Skip to content

tweak docs again#37

Merged
nabobalis merged 1 commit into
mainfrom
docs
Jul 13, 2022
Merged

tweak docs again#37
nabobalis merged 1 commit into
mainfrom
docs

Conversation

@nabobalis

Copy link
Copy Markdown
Member

This is how we need to do it to avoid the doc build from breaking.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #37 (7cb7b30) into main (882c741) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #37   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           10        10           
  Lines          133       133           
=========================================
  Hits           133       133           
Impacted Files Coverage Δ
hvpy/api_groups/jpeg2000/get_jp2_image.py 100.00% <ø> (ø)
hvpy/facade.py 100.00% <ø> (ø)

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 882c741...7cb7b30. Read the comment docs.

@nabobalis

Copy link
Copy Markdown
Member Author

I did not realize that the parameters were class attributes, so when we take the docs, it is incorrect. So we have to add this, to avoid the build failing.

@nabobalis
nabobalis merged commit 3716af2 into main Jul 13, 2022
@nabobalis
nabobalis deleted the docs branch July 13, 2022 14:05
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.

2 participants