-
Notifications
You must be signed in to change notification settings - Fork 275
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 Stylelint as checkstyle reporter #1307
Conversation
576c29f
to
63591e4
Compare
You need to merge in #1310 |
plugin/src/main/java/io/jenkins/plugins/analysis/warnings/StyleLint.java
Outdated
Show resolved
Hide resolved
Oic, thanks a bunch of the quick release. I'll merge it into this PR once it's merged into master 👍🏻 |
Codecov Report
@@ Coverage Diff @@
## master #1307 +/- ##
=========================================
Coverage 79.66% 79.66%
- Complexity 1442 1443 +1
=========================================
Files 248 249 +1
Lines 5522 5528 +6
Branches 425 425
=========================================
+ Hits 4399 4404 +5
- Misses 974 975 +1
Partials 149 149
Continue to review full report at Codecov.
|
There we go, checks are all green. |
Hey, would you mind cutting a release please? I'm looking forward to use the recent addition on ci.j 👀 Thanks in advance! |
The change proposed adds stylint as alias for the checkstyle reporter. Follow up to jenkinsci/analysis-model#812
We may need an update of analysis-mode plus its api here, I'm not too sure about it.