You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi! With other pytest plugins that I'm familiar with, the base tool name is shown in the reportinfo header for files. I think this helps a user to parse the output. For example, pytest-pylint shows:
However, pytest-mypy doesn't show "[mypy]", just the file path (tested with pytest-pylint 0.10.3 and pytest 8.3.3). I think this used to be shown, but this was lost in this commit. Or perhaps the add_marker() call should do this, but doesn't seem to; I haven't dug deeply into that.
Could this be restored please? I think all that is needed is to prepend "[mypy] " to the third item in the return tuple from MypyFileItem.reportinfo(), unless the add_marker machinery needs to be modified, and/or tests added. Happy to submit a PR if desired.
The text was updated successfully, but these errors were encountered:
Hey, thanks for reaching out!
Sure, we can restore that 👍 Moreover, let's add a way for customize the header so anyone can make it show whatever they want.
Hi! With other pytest plugins that I'm familiar with, the base tool name is shown in the reportinfo header for files. I think this helps a user to parse the output. For example, pytest-pylint shows:
However, pytest-mypy doesn't show "[mypy]", just the file path (tested with pytest-pylint 0.10.3 and pytest 8.3.3). I think this used to be shown, but this was lost in this commit. Or perhaps the
add_marker()
call should do this, but doesn't seem to; I haven't dug deeply into that.Could this be restored please? I think all that is needed is to prepend "[mypy] " to the third item in the return tuple from
MypyFileItem.reportinfo()
, unless theadd_marker
machinery needs to be modified, and/or tests added. Happy to submit a PR if desired.The text was updated successfully, but these errors were encountered: