Drop Python 2.7 support from CI and tox

[kalikiana]

Fixes poo#89206

[Martchus]

That's basically what I've suggested in the ticket so I'm fine with it.

[okurz]

I would be more explicit with dropping python2 support, not just remove it from testing but searching for all relevant references

[Martchus]

I wanted to write the same but couldn't find any references to Python 2 in the README.

[kalikiana]

The only mentions of it are in the Travis and tox configurations afair

[okurz]

well, it's a bit hidden but there, e.g.

• mentions of pip and package names in https://github.com/os-autoinst/openqa_review/blob/master/README.md#usage , probably need to be python3 variants
• https://github.com/os-autoinst/openqa_review/blob/master/setup.py#L12
• https://github.com/os-autoinst/openqa_review/blob/master/setup.py#L45
• https://github.com/os-autoinst/openqa_review/blob/master/openqa_review/openqa_review.py#L90

though all except the first is basically "compatibility code" which doesn't harm, so probably only the first point needs covering

[okurz]

tests fail now. As suggested in #140 (comment) please only handle the first point

[okurz]

have you seen my suggestion to only focus on "mentions of pip and package names in https://github.com/os-autoinst/openqa_review/blob/master/README.md#usage , probably need to be python3 variants"?

[kalikiana]

have you seen my suggestion to only focus on "mentions of pip and package names in https://github.com/os-autoinst/openqa_review/blob/master/README.md#usage , probably need to be python3 variants"?

Yes. And I was following your suggestion to drop the FileNotFoundError = IOError bit. But I'm splitting up the changes now and I would suggest to only keep the tox/ci bits and disambiguate documented commands in this PR.