Skip to content
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

Error Prone Parser does not detect warnings from maven builds #37

Open
uhafner opened this issue May 15, 2018 · 5 comments
Open

Error Prone Parser does not detect warnings from maven builds #37

uhafner opened this issue May 15, 2018 · 5 comments
Labels

Comments

@uhafner
Copy link
Contributor

uhafner commented May 15, 2018

Seems that maven uses a format that is not compatible with your parser:

[INFO] --- maven-compiler-plugin:3.7.0:compile (default-compile) @ analysis-model ---
[INFO] Changes detected - recompiling the module!
[INFO] Compiling 143 source files to /Users/hafner/Development/jenkins/workspace/Model - Freestyle - New/target/classes
[WARNING] /Users/hafner/Development/jenkins/workspace/Model - Freestyle - New/src/main/java/edu/hm/hafner/analysis/parser/RobocopyParser.java:[29,45] [StringSplitter] String.split(String) has surprising behavior
    (see http://errorprone.info/bugpattern/StringSplitter)
  Did you mean 'String file = matcher.group(4).split("\\s{11}", -1)[0];'?
[INFO] 

This warning is not picked up by the parser.
(Maven 3.5.2, violations-lib 1.54)

uhafner added a commit to jenkinsci/analysis-model that referenced this issue May 15, 2018
@tomasbjerre
Copy link
Owner

I'm actually regretting adding it as a supported tool. There seems to be many variations of how it prints the findindings in the log.

I opened an issue about parsable reports at their repo: google/error-prone#444

@uhafner
Copy link
Contributor Author

uhafner commented May 16, 2018

Yes, that makes sense. I added a vote for that issue.

@uhafner
Copy link
Contributor Author

uhafner commented Feb 5, 2019

BTW: your parser also can't parse Gradle output with Windows file letters, see jenkinsci/analysis-model@61b4883

@tomasbjerre
Copy link
Owner

google/error-prone#1132

@uhafner
Copy link
Contributor Author

uhafner commented Feb 20, 2019

Ah, nice! Hopefully they merge it soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants