On Tue, Jun 29, 2021 at 4:25 PM Wainer dos Santos Moschetta < waine...@redhat.com> wrote:
> Hi John, > > On 6/29/21 1:42 PM, John Snow wrote: > > Move it up near the check-pipenv help text, and update it to suggest > parity. > > > > (At the time I first added it, I wasn't sure if I would be keeping it, > > but I've come to appreciate it as it has actually helped uncover bugs I > > would not have noticed without it. It should stay.) > > > > Signed-off-by: John Snow <js...@redhat.com> > > Reviewed-by: Willian Rampazzo <willi...@redhat.com> > > --- > > python/Makefile | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/python/Makefile b/python/Makefile > > index 07ad73ccd0..d2cfa6ad8f 100644 > > --- a/python/Makefile > > +++ b/python/Makefile > > @@ -9,13 +9,17 @@ help: > > @echo " Requires: Python 3.6 and pipenv." > > @echo " Hint (Fedora): 'sudo dnf install python3.6 pipenv'" > > @echo "" > > + @echo "make check-tox:" > > + @echo " Run tests against multiple python versions." > > + @echo " These tests use the newest dependencies." > > + @echo " Requires: Python 3.6 - 3.10, and tox." > > + @echo " Hint (Fedora): 'sudo dnf install python3-tox > python3.10'" > > + @echo "" > > Somewhat related... in my system I don't have all supported python > versions installed, thus check-tox fails. > > Instead, maybe, you could configure tox (as below) to test to whatever > supported versions the developer have installed in the system; and on > absence of some versions it won't fail the tests entirely. > > diff --git a/python/setup.cfg b/python/setup.cfg > index e730f208d3..1db8aaf340 100644 > --- a/python/setup.cfg > +++ b/python/setup.cfg > @@ -123,6 +123,7 @@ multi_line_output=3 > > [tox:tox] > envlist = py36, py37, py38, py39, py310 > +skip_missing_interpreters=true > Didn't know this was an option, to be honest ... I wonder if it can be toggled on/off easily? I like the idea that it will fail if we don't set up the CI environment correctly instead of succeeding quietly. Though, you're right, some is better than none. Send a patch if you want? --js