* Flake8 is moved to github now.
* tox-py38 job is broken on opendev - it uses default nodeset, which
runs on ubuntu-jammy. But ubuntu-jammy does not have python 3.8,
and job fails.
Override it's nodeset to ubuntu-focal, which still has it.
Change-Id: Id9b20f48f112f659bbb4e891e73933b1c56250c1
Setting desired version of black to 22.3.0 prevents issues with
the click dependency described in the #1966879.
Introduction of black 22.3.0 forced style changes in following files:
- doc/source/conf.py
- jenkins_jobs/local_yaml.py
- tests/base.py
These changes do not modify behavior of the programs concerned.
Signed-off-by: Jiri Podivin <jpodivin@redhat.com>
Change-Id: I42a46f9ed31ea0f404d698eec73d30975e2e6f4c
Black dropped Python 2 support in 22.1.0. Unfortunately we can't pin
the version as <22.1.0 becasue in the past Black was versioned using
just main.major, not main.major.minor, and they were only releasing beta
versions, hence the 'b' suffix.
Pluggy 1.0.0 dropped Python 2 support.
It seams JJB should drop support for Python 2 too. Otherwise it will
be harder and harder to maintain dependencies...
Change-Id: I13b82a02bb50ab46183ff04aae521b44e475f743
Instead of having our reviewers spend time and effort commenting on
code style in changes let's make the process more automated by using
an opinionated tool like python-black.
Change-Id: Ia2724a0b38381a2ef1860c3fcd9d44bb51b8ddbb
Signed-off-by: Thanh Ha <zxiiro@gmail.com>
- upgrade flake8 to 3.6.0
- remove use of hacking since pre-commit replaced it
- mentioned minimal flake8 version on pre-commit config
- fixed newer linting errors
- disable W504 as it seems not possible to fix (conflict with W403)
- replaced buggy tox deps installation with native pip install, which
address the issue of missing to install new deps when the reqs files
are updated, developer being forced to to recreate the virtualenv.
- prepare for migration from tox-pep8 to tox-linters, now these being
just aliases.
Change-Id: I310578dce215aaf00a5b2d54716f90da9a1ecb4d
Depends-On: https://review.openstack.org/#/c/613726/
Vsevolod Fedorov