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

remove unsupported system_packages rtd config setting #109

Merged
merged 1 commit into from Sep 6, 2023

Conversation

braingram
Copy link
Collaborator

@braingram braingram commented Sep 6, 2023

RTD no longer supports any system_packages setting:
https://blog.readthedocs.com/drop-support-system-packages/

This PR removes the unused system_packages setting

@braingram braingram merged commit 1ade408 into spacetelescope:master Sep 6, 2023
17 checks passed
@braingram braingram deleted the fix_rtd branch September 6, 2023 14:47
@zacharyburnett
Copy link
Collaborator

good catch! I'll apply this to other repos as well

@braingram
Copy link
Collaborator Author

good catch! I'll apply this to other repos as well

Thanks!

stdatamodels (https://github.com/spacetelescope/stdatamodels) and jwst (spacetelescope/jwst#7860) should be all set.

@WilliamJamieson has an approved PR for roman_datamodels: spacetelescope/roman_datamodels#266
and a merged one for romancal: spacetelescope/romancal#853

rad doesn't appear to be effected: https://github.com/spacetelescope/rad/blob/main/.readthedocs.yml
stcal does: https://github.com/spacetelescope/stcal/blob/eba3b6a638e78c5d100bdf2da4dc1f787cfe05a2/.readthedocs.yaml#L25
beyond that I'm not sure.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-entry-needed does not require an entry in `CHANGES.rst`
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants