Github Action with Suggested Changes for PRs? #768
Replies: 5 comments
-
Might be that the third-party support on docs with tools like reviewdog should be handling this? EDIT: Found the feature in reviewdog as a recent beta development: reviewdog/reviewdog#678 |
Beta Was this translation helpful? Give feedback.
-
Suggested Changes is a nice feature, but 3rd-party like ReviewDog may do it. As baseline, setup-node support ESLint's stylish format.
It shows a lint error message on the PR. I agree that we need to add documentation about CI + textlint.
|
Beta Was this translation helpful? Give feedback.
-
I think the reviewdog integration might be the way to go for me, I spent some time looking for how others might have been using the two Github features together but it doesn't seem that common I guess. Once the Thanks for the links, I haven't looked at source, but there is a textlint Github Action for reviewdog, which AFAIK should mostly be what you'd want for a generic Github Action minus the extras to support reviewdog, likewise they have a similar example linked from the reviewdog README for Circle CI textlint that could be referenced. |
Beta Was this translation helpful? Give feedback.
-
We might want to add |
Beta Was this translation helpful? Give feedback.
-
Orthograph-err is a thin abstraction over TextLint which does exactly this. Currently it inserts annotations using the checks API, however using review comments to add suggestions for fixable items is an excellent idea. |
Beta Was this translation helpful? Give feedback.
-
I have been contributing to docs and often the review process might bring up common issues not only with style conventions which are more reliable to automate, but prose.
textlint
seems to be quite interesting with the latter, but probably better to have a Github Action that uses the Suggested Changes github feature to allow the author to conditionally apply the reported changes manually, or discuss with maintainers (with the maintainers potentially having less review suggestions to make).Unclear how that is setup, I've seen Github Actions for running linters and automatically applying fixes without the Suggested Changes feature used. If this is something being used already, it'd be great if the docs could cover this. I assume it'd need a custom formatter to wrap each change into the Suggested Changes codefence syntax, then send that output as a PR comment/review requesting changes?
Beta Was this translation helpful? Give feedback.
All reactions