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

Add .readthedocs.yaml #132

Merged
merged 5 commits into from
Aug 7, 2023
Merged

Add .readthedocs.yaml #132

merged 5 commits into from
Aug 7, 2023

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Aug 7, 2023

I got an email that RTD is deprecating a setting and noticed that pyorbital still doesn't have an explicit config which I think is also going to be required soon. This PR adds one.

  • Closes #xxxx
  • Tests added
  • Tests passed
  • Passes flake8 pyorbital
  • Fully documented

@djhoese djhoese self-assigned this Aug 7, 2023
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #132 (7b58f86) into main (1075142) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   87.50%   87.50%           
=======================================
  Files          14       14           
  Lines        2209     2209           
=======================================
  Hits         1933     1933           
  Misses        276      276           
Flag Coverage Δ
unittests 87.50% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@coveralls
Copy link

coveralls commented Aug 7, 2023

Coverage Status

coverage: 87.556%. remained the same when pulling 7b58f86 on doc-rtd-config into 1075142 on main.

@djhoese djhoese merged commit aa4c658 into main Aug 7, 2023
25 checks passed
@djhoese djhoese deleted the doc-rtd-config branch August 7, 2023 15:10
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.

2 participants