Switch away from iteration shims.

Review Request #11850 — Created Oct. 14, 2021 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

In the Python 2 to 3 conversion, many iteration methods (such as
items() and keys() were changed from returning lists to iterators.
In order to be correct in usage everywhere, we used six to ensure
consistent behavior across versions. Now that we're Python 3+ only on
the master branch, we can get rid of these and just use the builtins.

Ran unit tests.

Diff Revision 3 (Latest)

orig
1
2
3

Commits

First Last Summary ID Author
Switch away from iteration shims.
In the Python 2 to 3 conversion, many iteration methods (such as `items()` and `keys()` were changed from returning lists to iterators. In order to be correct in usage everywhere, we used `six` to ensure consistent behavior across versions. Now that we're Python 3+ only on the master branch, we can get rid of these and just use the builtins. Testing Done: Ran unit tests.
e62db00a4e531d577797df057de1f85ab4f67d1a David Trowbridge
This diff has been split across 3 pages: 1 2 3 >
contrib/internal/prepare-dev.py
contrib/internal/webapi-lint.py
reviewboard/tests.py
reviewboard/accounts/backends/standard.py
reviewboard/admin/security_checks.py
reviewboard/admin/forms/auth_settings.py
reviewboard/admin/forms/general_settings.py
reviewboard/admin/management/sites.py
reviewboard/avatars/services.py
reviewboard/avatars/templatetags/avatars.py
reviewboard/cmdline/rbsite.py
reviewboard/cmdline/tests/test_rbsite.py
reviewboard/datagrids/sidebar.py
reviewboard/diffviewer/diffutils.py
reviewboard/diffviewer/opcode_generator.py
reviewboard/diffviewer/parser.py
reviewboard/diffviewer/models/diffset.py
reviewboard/diffviewer/models/filediff.py
reviewboard/diffviewer/tests/test_commit_utils.py
reviewboard/diffviewer/tests/test_filediff.py
This diff has been split across 3 pages: 1 2 3 >
Loading...