redpen-cc / redpen

RedPen is an open source proofreading tool to check if your technical documents meet the writing standard. RedPen supports various markup text formats (Markdown, Textile, AsciiDoc, Re:VIEW, reStructuredText and LaTeX).
https://redpen.cc
Apache License 2.0
569 stars 74 forks source link

Question or Feature Request: redpen output plain with filepath for filename #869

Closed tsuyoshicho closed 4 years ago

tsuyoshicho commented 4 years ago

log:

see this log https://github.com/tsuyoshicho/action-redpen/pull/2/checks?check_run_id=368988899

Screenshot from Gyazo

This log is working with reviewdog.

File-link is dead.

reason:

redpen report plain

<file without path>:<line>:<message>

This format like Vim errorformat (e.g. grep,...)

But unavailable due to missing information and incomplete file name. lost path.

request:

I want to output with files path as an option or default.

Or if already have that feature, please let me know

tsuyoshicho commented 4 years ago

example filepath alive log tool: textlint

https://github.com/tsuyoshicho/action-textlint/pull/2/checks?check_run_id=349166176

takahi-i commented 4 years ago

Thank you very much for the feature request @tsuyoshicho. I will look into this issue and hopefully add some fix for the problem. In my understanding, the problem come from redpen plain format does not contains the file path.

tsuyoshicho commented 4 years ago

Yes, It is.

takahi-i commented 4 years ago

I see. I will give it a try 👍

takahi-i commented 4 years ago

I released version 1.10.4 https://github.com/redpen-cc/redpen/releases/tag/redpen-1.10.4 contains this patch. I close this issue. Please let us know when you still have the problem reopening this issue @tsuyoshicho.

tsuyoshicho commented 4 years ago

@takahi-i OK, but 1.10.4 attached file is not 1.10.4? re-check, please.

takahi-i commented 4 years ago

oops ... I changed the name of the package.

tsuyoshicho commented 4 years ago

Thanks quickly!!