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 extra-requirements from rtd config #5283

Merged
merged 1 commit into from
May 2, 2023

Conversation

bjlittle
Copy link
Member

🚀 Pull Request

Description

This PR mops up after #5262, and removes the now defunct PyPI docs optional dependencies reference within the .readthedocs.yml.


Consult Iris pull request check list

@codecov
Copy link

codecov bot commented Apr 27, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (948fb8a) 89.32% compared to head (6de8a9a) 89.32%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #5283   +/-   ##
=======================================
  Coverage   89.32%   89.32%           
=======================================
  Files          89       89           
  Lines       22390    22390           
  Branches     5374     5374           
=======================================
  Hits        20000    20000           
  Misses       1640     1640           
  Partials      750      750           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@bjlittle bjlittle force-pushed the rtf-no-pip-extra-requirements branch from 6a96176 to b10775a Compare April 29, 2023 14:42
@bjlittle bjlittle force-pushed the rtf-no-pip-extra-requirements branch from b10775a to 6de8a9a Compare April 29, 2023 21:57
Copy link
Contributor

@tkknight tkknight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The docs built ok on the PR so looks good.

@tkknight tkknight merged commit 9f821d0 into SciTools:main May 2, 2023
@bjlittle
Copy link
Member Author

bjlittle commented May 2, 2023

@tkknight Awesome, thanks 😄

@bjlittle bjlittle deleted the rtf-no-pip-extra-requirements branch May 2, 2023 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: 🏁 Done - v3.6.0
Development

Successfully merging this pull request may close these issues.

2 participants