-
Notifications
You must be signed in to change notification settings - Fork 280
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
Add support for Vale #1916
base: main
Are you sure you want to change the base?
Add support for Vale #1916
Conversation
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.
Thanks! I'll put this on hold until I am finished with the integration of the latest analyssi-model release...
(See #1917)
Sounds good. Thank you. |
So it tool a while but now the new release is available... Can you please rebase and fix the CheckStyle warning then this PR is ready as well! |
@uhafner seems like the checks did not rerun after my last push. Can you rerun the pipeline? |
I started the job again. However, I'm not sure if that helps or if I do have a bug in my GitHub pipeline so that it does not resolve the latest version (I had this problem in another repo as well recently) |
Yeah, seems like the pipeline is not pulling the latest commits. |
This adds parsing support for results produced by vale (https://vale.sh/)
Vale is a static checker for English prose.
Depends on jenkinsci/analysis-model#1119
Testing done
Junit test added.
We're also using this in our own infrastructure.
Submitter checklist