-
Notifications
You must be signed in to change notification settings - Fork 524
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
AO3-6605 Add Reviewdog for Rubocop check #4631
AO3-6605 Add Reviewdog for Rubocop check #4631
Conversation
According to |
@Bilka2 good point! I have gotten JShint checks on a PR in the past, so I think it still works (at least somewhat)? I will also update the instructions to not totally remove Hound yet |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good and works on my fork. The suggested change would be nice to have though.
Co-authored-by: Bilka <Bilka2@users.noreply.github.com>
I've added |
Pull Request Checklist
as the first thing in your pull request title (e.g.
AO3-1234 Fix thing
)until they are reviewed and merged before creating new pull requests.
Issue
https://otwarchive.atlassian.net/browse/AO3-6605
Purpose
Add a Reviewdog action to run Rubocop. This will give us more flexibility to add custom cops and upgrade to the latest version of rubcop (and its plugins). Example of how Rubocop warnings will show up: https://github.com/brianjaustin/ao3-sandbox/pull/1/files (the specific message is from a custom cop).
Note: no changes in what Rubocop flags are expected yet. Custom cops (and maybe other plugins?) will come in later PR(s)
Follow-Up Instructions
2. Deactivate/remove this repository from our Hound account -- optional but recommended to avoid duplicate comments and errors when we upgrade or add custom copsCredit
Brian Austin (they/he)