pytest.main: collect: factor out _visit_filter#4747
Conversation
| if six.PY2: | ||
|
|
||
| @staticmethod | ||
| def _visit_filter(f): |
There was a problem hiding this comment.
i recall this can be stored as constant using methodcaller('check', file=True, notext="pyc")
There was a problem hiding this comment.
Not sure if I understand correctly, but there is no "notext" in py.
There was a problem hiding this comment.
there is a ext checker method and the not prefix of the kwarg triggers inversion ^^
its horrendous in a funny way
There was a problem hiding this comment.
Would rather keep it like it is now (i.e. only move it), but feel free to push an update, or patch here.
There was a problem hiding this comment.
@RonnyPfannschmidt
What do you plan? Either post a patch/push a fixup (feel free to squash), or approve it, please.
There was a problem hiding this comment.
yikes, thanks for the call out, i missed that bit
No description provided.