@square-psychiatrist-19087 has this spiffy PR that will make all maintainers lives easier, and has been hanging out since March. I think overall at this point it makes the most sense to land and iterate, instead of stalling the PR. But @square-psychiatrist-19087 did the right thing and wrote tests and has thus stepped on a

conftest.py

rake that causes the tests to fail in ci. I've proposed on possible fix in the PR. But if anyone has opinions on how to handle conftest.py + resolves in this case please take a look. https://github.com/pantsbuild/pants/pull/22064

Trying to grok the issue

We set

--noskip

for pytest args in

pants.ci.toml

(for everthing), but that isn't a normal pytest thing, it's something Pants created (in the root

conftest.py

file) and is only part of

python-default

. So any python tests not in

python-default

fail with 'hey what is this

--noskip

arg

Yep, was just playing with it

hmm, I suppose it in part depends on how nice the

tqdm

dep is (it certainly looks nice) and then on how many new deps would accrue as the tool upgrade script expanded.

Thanks, I've applied the patch