On Pants 2 14 I m noticing some new Pants cannot infer owner

Question

On Pants 2 14 I m noticing some new Pants cannot infer owners for the following imports warnings One is for a dependency in a Python test file which is included in my ` pytest extra requirements` list and another is for a type that is included in ` mypy extra type stubs` whose import is gated behind an `if TYPE CHECKING` guard in the test file This worked without issue in 2 13 is this a bug or a legitimate new change in behavior If the latter what would be the recommended way to fix this Thanks

sparse-lifeguard-95737 · Answer

inference for extra type stubs has never worked the change in 2 14 is that the default behavior for dependency inference failures in general flipped from ignore to warn

sparse-lifeguard-95737 · Answer

if it s gated behind an `if TYPE CHECKING` then you can add a comment on the import line to explicitly disable dependency inference I think the warning messages you re seeing should include the comment

sparse-lifeguard-95737 · Answer

otherwise you can move your dependency from `extra type stubs` into your main requirements and mark it as a type module

brash-baker-91190 · Answer

Interesting is there special syntax for marking something as a type module

sparse-lifeguard-95737 · Answer

yes the `python requirements` target has a `type stubs module mapping` field <https www pantsbuild org docs reference python requirements codetype stubs module mappingcode>

sparse-lifeguard-95737 · Answer

if you list something there pants will know its a type stub module

sparse-lifeguard-95737 · Answer

Pants also guesses that anything starting ending with `types` or `stubs` is a stubs module

sparse-lifeguard-95737 · Answer

I actually have a PR open now to add this info to the docs <https github com pantsbuild pants pull 17076> if you want to take a look and let me know if anything is confusing missing sweat smile

brash-baker-91190 · Answer

Hrm I think I may have misunderstood something I had `pytest mock` in ` pytest extra requirements` which is what Pants was complaining about The documentation for that field suggests it s more for plugins that pytest itself needs rather than dependencies that the tests actually need

sparse-lifeguard-95737 · Answer

oh sorry misread because I was working on the mypy issue

sparse-lifeguard-95737 · Answer

but the underlying problem is the same the `python infer` subsystem can t see extra requirements fields in other subsystems

brash-baker-91190 · Answer

No you re good I also had a mypy issue sweat smile

sparse-lifeguard-95737 · Answer

for the pytest thing adding the comment suggested by the warning is the only work around I think

brash-baker-91190 · Answer

moving the `pytest mock` dependency into my main requirements file also appears to work