Skip to content

Commit 4c07b49

Browse files
feanilclaude
andcommitted
fix: pin setuptools<82 in static-assets-check workflow
setuptools 82+ removed pkg_resources, which pyfilesystem2 (fs) still uses for namespace package declarations. The constraints.txt pin handles this for full installs, but the static-assets-check workflow only installs requirements/edx/assets.txt. Pre-installing pip-tools.txt ensures setuptools 81.x is in place before the assets install runs. See: PyFilesystem/pyfilesystem2#577 Co-Authored-By: Claude Sonnet 4.6 <[email protected]>
1 parent 620567e commit 4c07b49

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

.github/workflows/static-assets-check.yml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,13 @@ jobs:
7171

7272
- name: Install Limited Python Deps for Build
7373
run: |
74+
# Install pip-tools.txt first to pin setuptools<82 before installing
75+
# assets.txt. setuptools 82+ removed pkg_resources, which pyfilesystem2
76+
# (fs) still uses for namespace package declarations. The constraints.txt
77+
# pin covers full installs, but this step only installs assets.txt so we
78+
# pre-install pip-tools.txt to ensure setuptools 81.x is in place.
79+
# See: https://github.com/PyFilesystem/pyfilesystem2/issues/577
80+
pip install -r requirements/pip-tools.txt
7481
pip install -r requirements/edx/assets.txt
7582
7683
- name: Add node_modules bin to $Path

0 commit comments

Comments
 (0)