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

chore: outdated transifex client cleanup #104

Merged
merged 1 commit into from
May 30, 2022

Conversation

edx-requirements-bot
Copy link
Contributor

@edx-requirements-bot edx-requirements-bot commented May 25, 2022

JIRA: https://2u-internal.atlassian.net/browse/BOM-3390

Description

This PR removes the outdated transifex client from the requirement files.
To Install the latest Transifex go client one can use the following script:
curl -o- https://raw.githubusercontent.com/transifex/cli/master/install.sh | bash
This script works on Linux and MacOS only

This pull request was generated by the cleanup-python-code Jenkins job, which ran
cd requirements; sed -i '/transifex-client/d' *.in; cd ..; make upgrade;

@aht007 aht007 changed the title chore: outdate transifex client cleanup chore: outdated transifex client cleanup May 26, 2022
@@ -103,17 +110,17 @@ edx-celeryutils==1.2.1
# via
# -r requirements/base.txt
# super-csv
edx-django-utils==4.5.0
edx-django-utils==5.0.0
Copy link
Member

Choose a reason for hiding this comment

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

https://github.com/openedx/edx-django-utils/releases/tag/5.0.0
Couldn't find the mentioned keys being used in this repo.

@@ -267,23 +281,22 @@ stevedore==3.5.0
# code-annotations
# edx-django-utils
# edx-opaque-keys
super-csv==2.1.4
super-csv==3.0.0
Copy link
Member

Choose a reason for hiding this comment

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

Support for Older Django versions was dropped. Nothing major

# via astroid
xblock==1.6.1
# via
# -r requirements/base.txt
# xblock-utils
xblock-utils==2.2.0
xblock-utils==3.0.0
Copy link
Member

Choose a reason for hiding this comment

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

Post Django32 cleanup

@aht007 aht007 marked this pull request as ready for review May 26, 2022 09:32
@aht007 aht007 force-pushed the jenkins/aht007/transifex-cleanup-cca9d4d branch from 17fdd04 to 632a101 Compare May 30, 2022 09:35
@aht007 aht007 merged commit 2b85d1e into master May 30, 2022
@aht007 aht007 deleted the jenkins/aht007/transifex-cleanup-cca9d4d branch May 30, 2022 09:36
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.

2 participants