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.

Sign up for GitHub

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

Jump to bottom

Post Django32 Cleanup #119

Merged
merged 1 commit into from
Feb 16, 2022
Merged

Post Django32 Cleanup #119

merged 1 commit into from
Feb 16, 2022

Conversation

edx-requirements-bot
Copy link
Contributor

Additional information from script execution

Python code cleanup by the cleanup-python-code Jenkins job.

This pull request was generated by the cleanup-python-code Jenkins job, which ran 
modernize_github_actions_django; modernize_github_actions; modernize_tox; modernize_setup_file; djcodemod run --removed-in 4.0 .;sed -i -E 's/django(<|>)[0-9]*\.[0-9]+/django<4.0/gi' ./requirements/constraints.txt; make upgrade;

The following packages were installed:
edx-repo-tools django-codemod

@edx-requirements-bot edx-requirements-bot requested review from mraarif and a team February 11, 2022 09:37
@aht007 aht007 force-pushed the jenkins/aht/POST-DJANGO32-CLEANUP-936c077 branch from 91de4c8 to 93e8504 Compare February 11, 2022 10:20
@aht007 aht007 marked this pull request as ready for review February 11, 2022 10:25
@aht007 aht007 force-pushed the jenkins/aht/POST-DJANGO32-CLEANUP-936c077 branch 2 times, most recently from a3a9e41 to db2268b Compare February 11, 2022 10:30
mraarif
mraarif reviewed Feb 14, 2022
View reviewed changes
tox.ini Outdated Show resolved Hide resolved
tox.ini Outdated Show resolved Hide resolved
@aht007 aht007 force-pushed the jenkins/aht/POST-DJANGO32-CLEANUP-936c077 branch from db2268b to e5b598b Compare February 14, 2022 14:00
@aht007 aht007 merged commit 7513e29 into master Feb 16, 2022
@aht007 aht007 deleted the jenkins/aht/POST-DJANGO32-CLEANUP-936c077 branch February 16, 2022 09:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mraarif mraarif mraarif left review comments

@UsamaSadiq UsamaSadiq UsamaSadiq approved these changes

@aht007 aht007 aht007 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@edx-requirements-bot @mraarif @UsamaSadiq @aht007