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.

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

Successfully merging this pull request may close these issues.

2 participants