[Tarantool-patches] [PATCH v3 0/5] Support Python 3 in tests and PEPify source code

Leonid Vasiliev lvasiliev at tarantool.org
Thu Dec 24 02:42:23 MSK 2020


Hi! Thank you for the patchset.
LGTM.

On 23.12.2020 15:36, sergeyb at tarantool.org wrote:
> From: Sergey Bronnikov <estetus at gmail.com>
> 
> Changes v3:
> - added patch with migration iteritems() to items()
> - revert changes that updates .result files
> - rework patch with bytes strings
> 
> Changes v2:
> - make using quotes with strings consistent
> - remove dead code
> - remove empty prints
> - added make target for running flake8
> - added flake8 config
> - added job to run flake8 on Gitlab CI
> 
> Changes v1:
> - convert print statements to functions
> - use "format()" instead of "%" everywhere
> 
> Gitlab CI: https://gitlab.com/tarantool/tarantool/-/pipelines/233768577
> Branch: https://github.com/tarantool/tarantool/tree/ligurio/gh-5538-support-python3
> Issue: https://github.com/tarantool/tarantool/issues/5538
> 
> Sergey Bronnikov (5):
>    test: convert print to function and make quotes use consistent
>    test: make dict.items() compatible with Python 3.x
>    test: make strings compatible with Python 3
>    test: get rid of iteritems()
>    test: remove dead code in Python tests end extra newlines
> 
>   test/app-tap/httpd.py                    |  44 ++---
>   test/box-py/args.test.py                 |  12 +-
>   test/box-py/bad_trigger.test.py          |  16 +-
>   test/box-py/bootstrap.test.py            |  16 +-
>   test/box-py/call.test.py                 |  80 ++++----
>   test/box-py/iproto.test.py               | 239 ++++++++++++-----------
>   test/box-py/print.test.py                |  25 +--
>   test/box-py/snapshot.test.py             |  19 +-
>   test/long_run-py/finalizers.test.py      |  10 +-
>   test/replication-py/cluster.test.py      | 217 ++++++++++----------
>   test/replication-py/conflict.test.py     |  55 +++---
>   test/replication-py/init_storage.test.py |  58 +++---
>   test/replication-py/multi.test.py        |  78 ++++----
>   test/replication-py/swap.test.py         |  55 +++---
>   test/xlog-py/big_lsn.test.py             |   4 +-
>   test/xlog-py/dup_key.test.py             |  23 ++-
>   test/xlog-py/empty.test.py               |   4 +-
>   test/xlog-py/lsn_gap.test.py             |  11 +-
>   test/xlog-py/misc.test.py                |  26 +--
>   test/xlog-py/missing.test.py             |  10 +-
>   20 files changed, 518 insertions(+), 484 deletions(-)
> 


More information about the Tarantool-patches mailing list