Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use suggestions? #28

Closed
rewritten opened this issue Feb 11, 2021 · 7 comments · Fixed by #48
Closed

Use suggestions? #28

rewritten opened this issue Feb 11, 2021 · 7 comments · Fixed by #48
Assignees
Labels
help wanted Extra attention is needed question Further information is requested

Comments

@rewritten
Copy link

I am adding code health checks to some of our repos, and I tried to use this action along with https://github.com/reviewdog/action-suggester. Of course it does not work, the suggester action expects to find a diff, not a linter output.

What would be the steps to support suggestions using this image? I can do the grunt work if I'm pointed in the right direction.

@mgrachev mgrachev added the question Further information is requested label Feb 12, 2021
@mgrachev
Copy link
Member

@haya14busa Can you tell us more about the suggest feature and how can we use it with this action?

@haya14busa
Copy link
Member

That's great to hear!

You need to support rdformat (https://github.com/reviewdog/reviewdog#code-suggestions) as rubocop output.

Example: eslint https://github.com/reviewdog/action-eslint/tree/master/eslint-formatter-rdjson

@AlexeyMatskevich
Copy link
Member

I will try to do this!

@mgrachev
Copy link
Member

I will try to do this!

Thank you. Assigned you 👍

@mgrachev
Copy link
Member

mgrachev commented Mar 6, 2021

@AlexeyMatskevich Do you have any progress?

@gagalago
Copy link

gagalago commented Aug 6, 2021

I suppose that this documentation can help to implement this feature:

@ohbarye
Copy link
Member

ohbarye commented Aug 7, 2021

I'm trying to write a custom Rubocop formatter that generates a JSON result with rdformat.
https://github.com/ohbarye/action-rubocop/blob/e15ee5162645c78e70f480bfcb42fea5b000113d/rdjson_formatter.rb

If I spend some more time I may be able to implement the suggestion feature though, should I leave this issue up to the assignee?

Suggestion example:

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants