Trying to grok the issue
# development
h
Trying to grok the issue
c
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
h
Yep, was just playing with it
So we won’t infer a dep on conftest.py because it’s in a different resolve
So yes, I think this is the right solution
and even if it’s not perfect, I don’t want the underlying change held up any longer
But I think it’s right
It’s a little clunky, but then so is that custom resolve in the first place (do we really need it?)
c
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.
s
Thanks, I've applied the patch
👀 1