Multiple reviews generated for multiline mypy error about action-mypy HOT 13 OPEN

Work in progress...

from action-mypy.

Hi,

I have added tsuyoshicho/test-action-mypy#2, but needs your approval to run CI.

For the initial sample error above, mypy can probably hide the note: lines without --show-error-context, therefore this time I created another kind of error that mypy cannot suppress.

from action-mypy.

Thanks a lot. (report and PR)

from action-mypy.

@tsuyoshicho OK it's done.

from action-mypy.

@abelcheung san

With no progress on JSON support, I've introduced a workaround.

You can ignore the notice of note.
Please try setting according to your needs.

Release 3.10.0 https://github.com/tsuyoshicho/action-mypy/releases/tag/v3.10.0
test result for ignore note: tsuyoshicho/action-test-repo#27

This issue remains open.

from action-mypy.

Hi @abelcheung san,

github.com/tsuyoshicho/test-action-mypy/pull/1
Simply reproduction PR

Basically, mypy report only error and work fine it.
If your project mypy.ini report note:, option mypy_flags use to suppress it.

If you have a problem that you haven't set up that way,
I want you to make a minimum reproduction PR and report it here. (like above PR)

from action-mypy.

As an example, following image shows the behavior I want to see -- multiple lines which are related to the same error can be grouped together.

from action-mypy.

FYI
https://github.com/jordemort/action-pyright
use rdjson format (rich error report support),
mypy json report python/mypy#10816

I will think about it.

from action-mypy.

Ouch, you're right, it is not easy to handle mypy error in its current state. It is fine to put the idea on hold until mypy has done something to improve the situation.

from action-mypy.

tsuyoshicho/action-test-repo#21

Reproduce PR create in my test repo.
This code is same as abel's test code.

@abelcheung san
Your PR are closed
Please remove forked test-action-mypy repository.
This original test-action-mypy repository are other user test repo...Sorry.
I remove my forked repository too.

from action-mypy.

Note:
python/mypy#10816 (comment)

FYI:
Relation File Format information.
reviewdog are diagnostics parsing tool, and that support json file.
https://github.com/reviewdog/reviewdog/tree/master/proto/rdf#rdjson

action-pyright using converted rdjson from original json.

When mypy implements Pyright's format json, converting the json using already existing action-pyright's method will be solved this issue.

from action-mypy.

@tsuyoshicho I have tried the ignore_note setting and it works fine here. It seems mypy developers don't have interest in implementing JSON support, so your workaround is probably the best approach right now.

from action-mypy.