From f7190cd61f5491d580aa0417a8ebafc6f4a375ab Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Wed, 4 Mar 2026 12:19:50 -0500 Subject: [PATCH 1/2] build: Fix the docs build. Our use of `fs` means we have a runtime dependency on setuptools. The newest version is not compatible with the current version of `fs` so we've added a constraint. But the docs build didn't previously install our specified version of setuptools. This is to fix that. --- .readthedocs.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/.readthedocs.yaml b/.readthedocs.yaml index fdef59eb56b2..8de9481c1654 100644 --- a/.readthedocs.yaml +++ b/.readthedocs.yaml @@ -10,6 +10,10 @@ sphinx: python: install: + # Need to install this to set the correct version of steuptools for now + # because it is needed by fs + # See https://github.com/openedx/openedx-platform/issues/38068 for details. + - requirements: "requirements/pip.txt" - requirements: "requirements/edx/doc.txt" - method: pip path: . From 9ade6356263aa496f4ec5c1710c6ce371c43bdb0 Mon Sep 17 00:00:00 2001 From: Feanil Patel Date: Wed, 1 Apr 2026 10:41:07 -0400 Subject: [PATCH 2/2] temp: Apply suggestions from code review Co-authored-by: Sarina Canelake --- .readthedocs.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.readthedocs.yaml b/.readthedocs.yaml index 8de9481c1654..eaa59d1f301a 100644 --- a/.readthedocs.yaml +++ b/.readthedocs.yaml @@ -10,7 +10,7 @@ sphinx: python: install: - # Need to install this to set the correct version of steuptools for now + # Need to install this to set the correct version of setuptools for now # because it is needed by fs # See https://github.com/openedx/openedx-platform/issues/38068 for details. - requirements: "requirements/pip.txt"