search

tomasbjerre / violations-maven-plugin

Maven plugin that will find report files from static code analysis, present and optionally fail the build.
Other
4 stars 1 forks source link

build failed after change README.md on Mac #5

Open sinkcup opened 2 years ago

sinkcup commented 2 years ago

steps:

  1. fork this project
  2. change and commit README.md
  3. run ./mwnw package

output:

[INFO] --- maven-surefire-plugin:2.12.4:test (default-test) @ violations-maven-plugin ---
[INFO] Surefire report directory: /Volumes/Code/ultimate-plan/jihu-gitlab-app/violations-maven-plugin/target/surefire-reports

-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running se.bjurr.violations.maven.plugin.ViolationCommentsMojoTest
Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.212 sec
Running se.bjurr.violations.maven.plugin.UpdateReadmeTest
Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.006 sec <<< FAILURE!
doUpdateReadmeWithReporters(se.bjurr.violations.maven.plugin.UpdateReadmeTest)  Time elapsed: 0.006 sec  <<< ERROR!
java.lang.RuntimeException: Could not find:

| Reporter | Parser | Notes
Missing a format? Open an issue [here](https://github.com/tomasbjerre/violations-lib/issues)!

 in ./README.md
    at se.bjurr.violations.lib.util.Utils.updateReadmeWithReporters(Utils.java:116)
tomasbjerre commented 2 years ago

Happens here: https://github.com/tomasbjerre/violations-lib/blob/f0d347d0c2a39fc26f2fd7bbf72b61fae0cb50c2/src/main/java/se/bjurr/violations/lib/util/Utils.java#L115

What did you change? The idea is to have the Violations auto-updated from: https://github.com/tomasbjerre/violations-lib/blob/f0d347d0c2a39fc26f2fd7bbf72b61fae0cb50c2/src/main/java/se/bjurr/violations/lib/reports/Reporter.java

sinkcup commented 2 years ago

I want to change nothing, but want to build it a maven package demo.