----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67059/ -----------------------------------------------------------
(Updated May 24, 2018, 2:45 p.m.) Review request for mesos, Andrew Schwartzmeyer, Benjamin Bannier, Eric Chung, and Kevin Klues. Changes ------- Changed dependency, improved README, and better pylint handling. Bugs: MESOS-8770 https://issues.apache.org/jira/browse/MESOS-8770 Repository: mesos Description ------- The scripts are in a temporary directory, support/python3. The scripts have been ported using 2to3, the official tool to do so. Many of these scripts require testing from the community before being used by default. The script building the virtual environment and the git hooks have been updated to use the new scripts if the environment variable `MESOSSUPPORTPYTHON` is set to `3` by the user. Diffs (updated) ----- support/README.md e812e9638ca555c56e4c52e52485240e75cb8229 support/build-virtualenv 850af89326721f34de20eb45a7e78fa391d031be support/hooks/post-rewrite 1ab14abf711d1923a7ae69beb33581317009a94a support/hooks/pre-commit 6faba98ab6db68aef1a54091a08b8db1eaac8701 support/python3/apply-reviews.py PRE-CREATION support/python3/generate-endpoint-help.py PRE-CREATION support/python3/jsonurl.py PRE-CREATION support/python3/mesos-gtest-runner.py PRE-CREATION support/python3/mesos-split.py PRE-CREATION support/python3/mesos-style.py PRE-CREATION support/python3/post-reviews.py PRE-CREATION support/python3/push-commits.py PRE-CREATION support/python3/test-upgrade.py PRE-CREATION support/python3/verify-reviews.py PRE-CREATION Diff: https://reviews.apache.org/r/67059/diff/4/ Changes: https://reviews.apache.org/r/67059/diff/3-4/ Testing ------- All the files are OK for our linter, I have tested `mesos-style.py` and `post-reviews.py` (used for this review request). We will likely see error messages `TypeError: cannot use a string pattern on a bytes-like object`, they are very easy to fix (we just need to add `.decode("utf-8")` but detecting all of them requires to use the scripts. Please do so and create issues. Thanks, Armand Grillet