Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Setting the version to match reality. Sym linking py3 to python for parity #135

Closed
wants to merge 1 commit into from

Conversation

coryleeio
Copy link
Contributor

No description provided.

Copy link
Contributor

@adzuci adzuci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@@ -19,10 +19,13 @@ MAINTAINER [email protected]
# MySQL-python for performance gains.

# If you add a package here please include a comment above describing what it is used for
RUN apt-get update && apt-get upgrade -qy && apt-get install language-pack-en locales git python3.5 python3-pip libmysqlclient-dev libssl-dev python3-dev -qy && \
RUN apt-get update && apt-get upgrade -qy && apt-get install language-pack-en locales git python3.6 python3-pip libmysqlclient-dev libssl-dev python3.6-dev -qy && \
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@coryleeio Do you know if we have tested 3.6 or what is needed to get this PR merged.

@sarina
Copy link
Contributor

sarina commented Nov 13, 2023

Closing as @coryleeio has left the Open edX project. The PR can be reopened if work should continue.

@sarina sarina closed this Nov 13, 2023
@openedx-webhooks openedx-webhooks added the open-source-contribution PR author is not from Axim or 2U label Nov 13, 2023
@openedx-webhooks
Copy link

Although this pull request is already closed, I'm still watching it for updates.

There is nothing you have to do. No action is needed from your side. Thanks.

1 similar comment
@openedx-webhooks
Copy link

Although this pull request is already closed, I'm still watching it for updates.

There is nothing you have to do. No action is needed from your side. Thanks.

@sarina sarina removed the open-source-contribution PR author is not from Axim or 2U label Nov 13, 2023
@nedbat nedbat deleted the coryleeio/dockerfile branch November 13, 2023 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants