tutor/tutor-discovery
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: wrong variable name for extra_requirement for pip extra requirements

Browse Source 
Co-authored-by: Muhammad Faraz  Maqsood <faraz.maqsood@a006-01130.home>
This commit is contained in:
Muhammad Faraz Maqsood 2023-11-08 11:50:17 +05:00 committed by GitHub
parent 4e90634092
commit 811e792a17
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
changelog.d/20231102_142414_faraz.maqsood_fix_extra_requirements_issue.md Normal file
View File

@ -0,0 +1 @@
- [BugFix] Corrected variable name for installing extra pip requirements. (by @Faraz32123)

2
tutordiscovery/templates/discovery/build/discovery/Dockerfile
View File

@ -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