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

Change original_output in RdjsonFormatter, adding file #80

Merged

Conversation

Slike9
Copy link
Contributor

@Slike9 Slike9 commented Jan 10, 2023

original_output field in RdjsonFormatter is printed by reviewdog on running the action with github-pr-review reporter.

Before, it did not contain file, thus messages were not fully informative.
Example https://github.com/Slike9/GitHub-Actions-PlayGround/actions/runs/3887880250/jobs/6634622915#step:5:61

C:  1:  1: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

This PR fixes the field, adding file.
Example https://github.com/Slike9/GitHub-Actions-PlayGround/actions/runs/3887873154/jobs/6634608438#step:5:61

index.rb:1:1: C: Style/FrozenStringLiteralComment: Missing frozen string literal comment.

This format is borrowed from ClangStyleFormatter formatter (default formatter, ProgressFormatter, inherits it).

Change `original_output`, adding file.

The field is printed by reviewdog on running the action with
github-pr-review reporter.

Before, it did not contain file, thus messages were not fully informative. E.g.
```
C: 30: 54: Rails/AttributeDefaultBlockValue: Pass method in a block to `:default` option.
```

This change fixes the field, adding file:
```
app/models/user.rb:30:54: C: Rails/AttributeDefaultBlockValue: Pass method in a block to `:default` option.
```
@mgrachev mgrachev merged commit eb901a2 into reviewdog:master Jan 11, 2023
@mgrachev
Copy link
Member

@Slike9 Thank you 👍

@github-actions
Copy link
Contributor

🚀 [bumpr] Bumped! New version:v2.3.2 Changes:v2.3.1...v2.3.2

@review-dog
Copy link
Member

Hi, @Slike9! We merged your PR to reviewdog! 🐶
Thank you for your contribution! ✨

We just invited you to join the @reviewdog organization on GitHub.
Accept the invite by visiting https://github.com/orgs/reviewdog/invitation.
By joining the team, you'll be a part of reviewdog community and can help the maintenance of reviewdog.

Thanks again!

@Slike9 Slike9 deleted the fix-rdjson-formatter-original-output branch January 11, 2023 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants