We're updating the issue view to help you get more done. 

Run python-modernize on edx-platform (327 of 380)

Description

Help prepare edx-platform for python3 by doing the following:

  1. Comment on the ticket to indicate that you are starting work on it.

  2. Launch a development shell in a Docker container via either "make shell" or devstack's "make lms-shell". Alternatively, you can create a new virtualenv or conda environment and install modernize and isort into it (this is a better choice on Windows).

  3. run

    1 python-modernize -w lms/djangoapps/course_api/blocks/urls.py lms/djangoapps/course_api/blocks/serializers.py lms/djangoapps/course_api/blocks/permissions.py lms/djangoapps/course_api/blocks/api.py lms/djangoapps/course_api/blocks/__init__.py lms/djangoapps/course_api/blocks/forms.py lms/djangoapps/course_api/blocks/views.py lms/djangoapps/course_api/tests/__init__.py lms/djangoapps/course_api/urls.py lms/djangoapps/course_api/serializers.py
  4. run

    1 isort -rc lms/djangoapps/course_api/blocks/urls.py lms/djangoapps/course_api/blocks/serializers.py lms/djangoapps/course_api/blocks/permissions.py lms/djangoapps/course_api/blocks/api.py lms/djangoapps/course_api/blocks/__init__.py lms/djangoapps/course_api/blocks/forms.py lms/djangoapps/course_api/blocks/views.py lms/djangoapps/course_api/tests/__init__.py lms/djangoapps/course_api/urls.py lms/djangoapps/course_api/serializers.py
  5. Make sure the changes look reasonable and submit them as a pull request; mention this ticket in the description and include INCR-<Ticket Number> in the name.

  6. Ask for tests to be triggered if they don't start automatically.

  7. Diagnose any test failures caused by the changes, and either fix them or ask for help.

  8. If you run into unexpected errors, see this document for common problems: https://openedx.atlassian.net/wiki/spaces/AC/pages/977666218/Using+python-modernize+effectively

Status

Assignee

Ayub Khan

Reporter

Feanil Patel

Labels

None

Epic Link

Priority

Unset