Skip to content
This repository has been archived by the owner on Aug 1, 2024. It is now read-only.

fix: update .readthedocs.yaml #1283

Merged
merged 1 commit into from
Mar 1, 2024
Merged

Conversation

robrap
Copy link
Contributor

@robrap robrap commented Feb 27, 2024

Attempt to fix the build. It is unclear why this is failing, so simply attempting to match the following as closely as possible:
https://docs.readthedocs.io/en/stable/config-file/v2.html

The failure no longer has an error (which used to be around missing os), and the raw output is missing the call:

cat .readthedocs.yaml

It's unclear what is breaking. I'm attempting to clean-up the redundant python version reference to see what happens.

Additional Context:


I've completed each of the following or determined they are not applicable:

  • Made a plan to communicate any major developer interface changes (or N/A)

@robrap robrap force-pushed the robrap/attempt-fix-readthedocs branch 2 times, most recently from f57b605 to 6fb427c Compare February 27, 2024 17:08
@robrap robrap force-pushed the robrap/attempt-fix-readthedocs branch from 6fb427c to f18e4e3 Compare February 27, 2024 17:16
@robrap robrap force-pushed the robrap/attempt-fix-readthedocs branch from f18e4e3 to e8c05a1 Compare March 1, 2024 21:06
Attempt to fix the build. It is unclear why this is failing,
so simply attempting to match the following as closely
as possible:
https://docs.readthedocs.io/en/stable/config-file/v2.html

The failure no longer has an error (which used to be around
missing os), and the raw output is missing the call:
> cat .readthedocs.yaml

It's unclear what is breaking.
@robrap robrap force-pushed the robrap/attempt-fix-readthedocs branch from e8c05a1 to eaa20c4 Compare March 1, 2024 21:09
Copy link
Contributor

@timmc-edx timmc-edx left a comment

Choose a reason for hiding this comment

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

I don't think this will fix the problem we're seeing on builds for master, but I don't think it will make anything worse, and maybe RTD was just having a transient problem.

@robrap robrap merged commit 323b475 into master Mar 1, 2024
5 of 12 checks passed
@robrap robrap deleted the robrap/attempt-fix-readthedocs branch March 1, 2024 21:23
@robrap
Copy link
Contributor Author

robrap commented Mar 1, 2024

I don't think this will fix the problem we're seeing on builds for master, but I don't think it will make anything worse, and maybe RTD was just having a transient problem.

I didn't think so either, but I wanted to rule this out, and it looks like it worked. I'll create a support ticket with readthedocs, but I'm not sure who gets those emails yet.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants