fix: wrong variable name for extra_requirement for pip extra requirem…

…ents Co-authored-by: Muhammad Faraz Maqsood <[email protected]>
overhangio · Nov 8, 2023 · 811e792 · 811e792
1 parent 4e90634
commit 811e792
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
@@ -47,7 +47,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)