Skip to content

Commit

Permalink
👷(edx-platform) fix build following missing Python package
Browse files Browse the repository at this point in the history
The moto Python package is missing but it is only used in tests
which we are no intent to run on this image. Let's remove it.
  • Loading branch information
sampaccoud committed Oct 12, 2023
1 parent b43a0de commit 4a1bad2
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 0 deletions.
4 changes: 4 additions & 0 deletions releases/dogwood/3/fun/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@ release.
- Upgrade fun-apps to version 5.16.0 to add a title to the course run
synchronization webhook data

### Fixed

- Remove dependency to missing Python package `moto` only used in tests

## [dogwood.3-fun-2.9.1] - 2023-07-07

### Fixed
Expand Down
2 changes: 2 additions & 0 deletions releases/dogwood/3/fun/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -153,6 +153,8 @@ COPY patches/* /tmp/
# Patches pre-install
# Patch strftime to fix bug when year is less than 1900
RUN patch -p1 < /tmp/edx-platform_dogwood.3-fun_strftime-1900.patch
# Patch to fix installation of Python modules
RUN patch -p1 < /tmp/edx-platform_dogwood.3-fun_moto.patch

# Install Javascript requirements
RUN npm install
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,14 @@
diff --git a/requirements/edx/base.txt b/requirements/edx/base.txt
index 19c655d79e..457d719e30 100644
--- a/requirements/edx/base.txt
+++ b/requirements/edx/base.txt
@@ -140,7 +140,7 @@ flaky==2.4.0
freezegun==0.1.11
mock-django==0.6.9
mock==1.0.1
-moto==0.3.1
+#moto==0.3.1 version not available anymore but we don't need to run the tests anyway
nose==1.3.7
nose-exclude
nose-ignore-docstring

0 comments on commit 4a1bad2

Please sign in to comment.