-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Bump reviewdog version to 0.20.1 #131
Comments
Yes, I'll update when I get a chance (PRs are also welcome). You can also currently change the version of reviewdog by providing a value for |
Cool, I'll give it a try then ! Thanks for confirming that you are not against this changes :) |
I'm having a go at this request, and would appreciate feedback on the PR when possible. Does it make sense to re-add the |
Hi, I was wondering if this action could use a more recent version of
reviewdog
.Currently, the version used is
0.17
vale-action/src/input.ts
Line 61 in 91ac403
while the most recent version is currently
0.20.1
One of the gain from changing the version would be to leverage
-fail-level
instead of-fail-on-error
.I don't know if you are open to the idea of adding back the
level
input which was removed previously (c4213d4). Since I couldn't find information on why it got removed, maybe that one of the reason for such limitation was the lack of feature inreviewdog
and using the new version might help on this.More information:
The text was updated successfully, but these errors were encountered: