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

MAINT: Update .readthedocs.yml to latest version in astropy package-template #609

Merged
merged 3 commits into from
Nov 18, 2023

Conversation

rrjbca
Copy link
Contributor

@rrjbca rrjbca commented Oct 29, 2023

Description

Readthedocs is currently failing for #608 with Problem in your project's configuration. Invalid configuration option "build.os": os not found: https://readthedocs.org/projects/skypy/builds/22378505/

This PR updates .readthedocs.yml following the latest version in astropy package-template which configures the build os: https://github.com/astropy/package-template/blob/cookiecutter/%7B%7B%20cookiecutter.package_name%20%7D%7D/.readthedocs.yml

Also fixes two subsequent failures:

  • Fixes incorrect underline length in the docstring for logistic_completeness_function
  • Add graphviz to apt_packages in readthecods configuration

Checklist

  • Follow the Contributor Guidelines
  • Write unit tests
  • Write documentation strings
  • Assign someone from your working team to review this pull request
  • Assign someone from the infrastructure team to review this pull request

@rrjbca rrjbca requested a review from a team October 29, 2023 08:59
@rrjbca rrjbca merged commit dd62ba5 into skypyproject:main Nov 18, 2023
8 checks passed
@rrjbca rrjbca deleted the update_readthedocs branch November 18, 2023 12:05
rrjbca added a commit that referenced this pull request Nov 19, 2023
…emplate (#609)

* Update .readthedocs.yml to latest version in astropy package-template

* Fix underline length

* Add graphviz to apt_packages
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.

1 participant