Skip to content

Conversation

@kazk
Copy link
Member

@kazk kazk commented Apr 16, 2021

  • Remove autoformatting action for now
  • Try reviewdog, starting with spell checker with misspell

```c
//declaration of solution accepting a two-dimentional array
//declaration of solution accepting a two-dimentional array.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[misspell] reported by reviewdog 🐶
"dimentional" is a misspelling of "dimensional"

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changed files get review comments like this. Everything else gets check annotation, and those are visible in "Checks" tab or each commit view.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

@kazk
Copy link
Member Author

kazk commented Apr 16, 2021

https://github.com/reviewdog/action-suggester can post suggestion comments based on autocorrection by tools.

@kazk kazk force-pushed the chore/try-reviewdog branch 2 times, most recently from f76b35c to 24c7911 Compare April 17, 2021 00:10
@kazk kazk force-pushed the chore/try-reviewdog branch from 24c7911 to e556631 Compare April 17, 2021 01:35
@kazk kazk force-pushed the chore/try-reviewdog branch from e556631 to f151c46 Compare April 17, 2021 01:36
@kazk kazk changed the title Experiment with Review Automation Use reviewdog to automate reviews Apr 17, 2021
@kazk kazk merged commit 181fde2 into master Apr 17, 2021
@kazk kazk deleted the chore/try-reviewdog branch April 17, 2021 01:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants