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.

Sign up for GitHub

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

Jump to bottom

Add reporter for outputing Reviewdog JSONL format #377

Closed
wants to merge 2 commits into from
Closed

Add reporter for outputing Reviewdog JSONL format #377

wants to merge 2 commits into from

Commits on Jul 12, 2021

  1. Add reporter for outputing Reviewdog JSONL format 

    This reporter will convert linter output to a standardised format that can
easily be read by reviewdog.

I have decided to use the JSONL format over the JSON format as it will be
simpler to join together mutltiple files before feeding into review dog.

Install reviewdog
To use with the reviewdog reporter we install reviewdog in the
docker image. This intentionally pinned to a specific version
to avoid issues if breaking changes occur in Reviewdog.

Add overall reviewdog reporter
This gathers results generated by the Reviewdog Linter Reporters
and feeds those results into reviewdog. This will make reviewdog
post comments to github as a PR review.

Configure pylint for reviewdog reporter

add unit tests for reviewdog reporter

Reviewdog: Checkstyle config

Dont run reviewdog reporter by default
    @ewencluley
    ewencluley committed Jul 12, 2021
    Configuration menu
    Copy the full SHA
    193d84d View commit details
    Browse the repository at this point in the history

  2. add reviewdog for javascript linters

    @ewencluley
    ewencluley committed Jul 12, 2021
    Configuration menu
    Copy the full SHA
    80decf8 View commit details
    Browse the repository at this point in the history