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.

Sign up for GitHub

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

Jump to bottom

update ReadTheDocs configuration file #148

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

update ReadTheDocs configuration file #148

wants to merge 2 commits into from

Conversation

zacharyburnett
Copy link
Collaborator

readthedocs.yml and the V1 of the configuration file is now deprecated, according to RTD docs

@sean-lockwood
Copy link
Member

sean-lockwood commented Jan 23, 2023 •
edited

I had to do some roundabout things in setup.py to get a reduced environment on RTD (see PR #141 ): setup.py diff

Do you know how to skip the setup.py install_requires dependencies in v2 and specify them in the doc/environment.yml file only?

Edit: I just saw your comment on #141. Will try that out.

starphire

This comment was marked as duplicate.

Sign in to view

@starphire starphire dismissed their stale review March 10, 2023 16:48

Approved wrong PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@starphire starphire starphire left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@zacharyburnett @sean-lockwood @starphire