Merge remote-tracking branch 'origin/master' into nightly

overhangio · Nov 8, 2023 · 16e90e1 · 16e90e1
2 parents b9dd940 + 811e792
commit 16e90e1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md b/changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md
@@ -0,0 +1 @@
+- [BugFix] Corrected variable name for installing extra pip requirements. (by @Faraz32123)
diff --git a/tutordiscovery/templates/discovery/build/discovery/Dockerfile b/tutordiscovery/templates/discovery/build/discovery/Dockerfile
@@ -48,7 +48,7 @@ ENV PATH /openedx/nodeenv/bin:${PATH}
 # instructions to benefit from docker image caching
 # Install base requirements
 RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install -r requirements.txt
-{% for extra_requirement in DISCOVERY_EXTRA_PIP_REQUIREMENTS %}RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install '{{ extra_requirements }}'
+{% for extra_requirement in DISCOVERY_EXTRA_PIP_REQUIREMENTS %}RUN {% if is_buildkit_enabled() %}--mount=type=cache,target=/openedx/.cache/pip,sharing=shared {% endif %}pip install '{{ extra_requirement }}'
 {% endfor %}
 
 # Install npm, bower requirements
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		- [BugFix] Corrected variable name for installing extra pip requirements. (by @Faraz32123)