elParaguayo / qtile-extras

Somewhere to store things I create for qtile that probably won't end up in the official repo
MIT License
166 stars 21 forks source link

The pytest_lazyfixture is not compatible with pytest 8 #388

Open FrostyX opened 2 days ago

FrostyX commented 2 days ago

We use the pytest_lazyfixture here:

$ grep -r lazy_fixture
test/widget/test_currentlayouticon.py:from pytest_lazyfixture import lazy_fixture
test/widget/test_currentlayouticon.py:@pytest.mark.parametrize("currentlayout_manager", [lazy_fixture("temp_icons")], indirect=True)

It is not compatible with pytest 8, we should probably use https://github.com/dev-petrov/pytest-lazy-fixtures instead.

elParaguayo commented 2 days ago

Yes - so we currently force pytest < 8.0.0. If that link now works, do you want to submit a PR with this?

FrostyX commented 2 days ago

I can submit a PR. So far, I am trying to get this into the official Fedora repositories, so I am filing all the upstream issues that I need. But after that, I can try to solve them :-)