Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-68643

Error prone parser does not work anymore

XMLWordPrintable

      17:16:14  /home/jenkins/agent/workspace/Foo.java:[32,17] [UnnecessaryParentheses] These grouping parentheses are unnecessary; it is unlikely the code will be misinterpreted without them
17:16:14      (see https://errorprone.info/bugpattern/UnnecessaryParentheses)
17:16:14    Did you mean '.randomize( FieldPredicates.named("absoluteRateLimit")'?

      Using error prone 2.13 as java maven apt processor the output seems to have changed so that the error prone parser does not detect any messages anymore.

          [JENKINS-68643] Error prone parser does not work anymore

          Michael Düsterhus created issue -
          Ulli Hafner made changes -
          Assignee Original: Ulli Hafner [ drulli ]
          Labels New: help-wanted newbie-friendly

          Ulli Hafner added a comment -

          I'm using 1.14 without any problems:

          [2022-09-22T15:50:55.456Z] [WARNING] /var/data/workspace/analysis-pipeline/src/test/java/edu/hm/hafner/analysis/ReportTest.java:[127,18] [InitializeInline] Initializing variables in their declaring statement is clearer, where possible.
[2022-09-22T15:50:55.456Z]     (see https://errorprone.info/bugpattern/InitializeInline)
[2022-09-22T15:50:55.456Z]   Did you mean to remove this line?

           It looks like that there is no WARNING printed in your case in Maven. You can check why this is different in your version (here is my log file: https://ci.jenkins.io/job/Plugins/job/analysis-model/job/master/lastBuild/console).

          Another approach would be to make the tag optional in https://github.com/jenkinsci/analysis-model/blob/master/src/main/java/edu/hm/hafner/analysis/parser/ErrorProneParser.java#L27.

          Ulli Hafner added a comment - I'm using 1.14 without any problems: [2022-09-22T15:50:55.456Z] [WARNING] /var/data/workspace/analysis-pipeline/src/test/java/edu/hm/hafner/analysis/ReportTest.java:[127,18] [InitializeInline] Initializing variables in their declaring statement is clearer, where possible. [2022-09-22T15:50:55.456Z] (see https://errorprone.info/bugpattern/InitializeInline) [2022-09-22T15:50:55.456Z] Did you mean to remove this line?  It looks like that there is no WARNING printed in your case in Maven. You can check why this is different in your version (here is my log file: https://ci.jenkins.io/job/Plugins/job/analysis-model/job/master/lastBuild/console ). Another approach would be to make the tag optional in https://github.com/jenkinsci/analysis-model/blob/master/src/main/java/edu/hm/hafner/analysis/parser/ErrorProneParser.java#L27 .
          Ulli Hafner made changes -
          Labels Original: help-wanted newbie-friendly New: hacktoberfest help-wanted newbie-friendly
          Ulli Hafner made changes -
          Resolution New: Cannot Reproduce [ 5 ]
          Status Original: Open [ 1 ] New: Closed [ 6 ]

            Unassigned Unassigned
            reitzmichnicht Michael Düsterhus
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: