mirror of
https://salsa.debian.org/freeipa-team/freeipa.git
synced 2024-12-22 23:23:30 -06:00
Constrain pylint to supported versions
Two, three times a year PR-CI starts failing because tox tests pull in a newer version of pylint with new warnings. To reduce breakage this change constraints pylint (and indirectly astroid) to latest tested minor version. The constraint should be updated when FreeIPA starts to support a new Fedora version with more recent pylint. Related: https://pagure.io/freeipa/issue/8818 Signed-off-by: Christian Heimes <cheimes@redhat.com> Reviewed-By: Stanislav Levin <slev@altlinux.org> Reviewed-By: Rob Crittenden <rcritten@redhat.com>
This commit is contained in:
parent
d2ca791549
commit
6d0d1c7088
@ -9,5 +9,7 @@ ipapython == @VERSION@
|
||||
ipaserver == @VERSION@
|
||||
ipatests == @VERSION@
|
||||
|
||||
# upstream pylint 1.7.5 fixed bad python3 import of stat module
|
||||
pylint >= 1.7.5
|
||||
# keep pylint version in sync with current Fedora release
|
||||
# F34 has 2.6
|
||||
# https://pagure.io/freeipa/issue/8818 added pylint 2.8 support
|
||||
pylint < 2.9
|
||||
|
@ -347,12 +347,8 @@ BuildRequires: python3-polib
|
||||
BuildRequires: python3-pyasn1
|
||||
BuildRequires: python3-pyasn1-modules
|
||||
BuildRequires: python3-pycodestyle
|
||||
%if 0%{?fedora} || 0%{?rhel} > 8
|
||||
# https://bugzilla.redhat.com/show_bug.cgi?id=1648299
|
||||
BuildRequires: python3-pylint >= 2.1.1-2
|
||||
%else
|
||||
BuildRequires: python3-pylint >= 1.7
|
||||
%endif
|
||||
# .wheelconstraints.in limits pylint version in Azure and tox tests
|
||||
BuildRequires: python3-pylint
|
||||
BuildRequires: python3-pytest-multihost
|
||||
BuildRequires: python3-pytest-sourceorder
|
||||
BuildRequires: python3-qrcode-core >= 5.0.0
|
||||
|
@ -2,5 +2,5 @@ steps:
|
||||
- script: |
|
||||
set -e
|
||||
sudo dnf -y install python3-pip
|
||||
python3 -m pip install --user --ignore-installed pylint
|
||||
python3 -m pip install --user --constraint .wheelconstraints.in --ignore-installed pylint
|
||||
displayName: Install Lint dependencies
|
||||
|
Loading…
Reference in New Issue
Block a user