Skip to content

tweak docs again#37

Merged
nabobalis merged 1 commit intomainfrom
docs
Jul 13, 2022
Merged

tweak docs again#37
nabobalis merged 1 commit intomainfrom
docs

Conversation

@nabobalis
Copy link
Member

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

@codecov-commenter
Copy link

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
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