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

Replace Codenarc parser with native parser

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • warnings-plugin
    • None 

              parsers.add(new ViolationsAdapter(new CodenarcParser(),
                Messages._Warnings_Codenarc_ParserName(),
                Messages._Warnings_Codenarc_LinkName(),
                Messages._Warnings_Codenarc_TrendName()));

          [JENKINS-17787] Replace Codenarc parser with native parser

          Pedro Furlanetto added a comment -

          Is there any update?

          Pedro Furlanetto added a comment - Is there any update?

          Ulli Hafner added a comment -

          No one is working on this issue up to now. Interested?

          Ulli Hafner added a comment - No one is working on this issue up to now. Interested?

          Pedro Furlanetto added a comment -

          Actually I am interested in using this in a project but I am not familiar with the work that has to be done here. If you give me some information and details, I can try during the weekends (I cannot promise anything )

          Pedro Furlanetto added a comment - Actually I am interested in using this in a project but I am not familiar with the work that has to be done here. If you give me some information and details, I can try during the weekends (I cannot promise anything )

          Ulli Hafner added a comment -

          Actually there is already a parser in the violations lib. I think the parsers of the violations lib can be included in the warnings plugin in the same way as I included the violations plugin parsers. There might be some things that need to be changed in the violations-lib though. E.g. the parsers should be serializable. I also did not yet check if the API could be adapted, since the warnings plugin works on an InputStream and the violations-lib on a String.

          Ulli Hafner added a comment - Actually there is already a parser in the violations lib . I think the parsers of the violations lib can be included in the warnings plugin in the same way as I included the violations plugin parsers. There might be some things that need to be changed in the violations-lib though. E.g. the parsers should be serializable. I also did not yet check if the API could be adapted, since the warnings plugin works on an InputStream and the violations-lib on a String.

          Pedro Furlanetto added a comment -

          I will start to work with this using my free time and if I have some question, I will let you know. Thanks

          Pedro Furlanetto added a comment - I will start to work with this using my free time and if I have some question, I will let you know. Thanks

          Ulli Hafner added a comment -

          Fine, thanks!

          Ulli Hafner added a comment - Fine, thanks!

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/AnalysisAction.java
          src/test/java/plugins/WarningsPluginTest.java
          src/test/resources/warnings_plugin/WarningMain.java
          src/test/resources/warnings_plugin/WarningMain2.java
          src/test/resources/warnings_plugin/jenkins-17787/CodeNarcReport2.xml
          src/test/resources/warnings_plugin/jenkins-17787/CodeNarcXmlReport.xml
          src/test/resources/warnings_plugin/out.txt
          http://jenkins-ci.org/commit/acceptance-test-harness/572a0e4dd79d457fd05a9c6a540f419711d1ac0e
          Log:
          Merge pull request #332 from stefanSchuhbaeck/warningsPlugin

          Warnings plugin Issue JENKINS-17787 with docker

          Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/876baae68246...572a0e4dd79d

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/org/jenkinsci/test/acceptance/plugins/analysis_core/AnalysisAction.java src/test/java/plugins/WarningsPluginTest.java src/test/resources/warnings_plugin/WarningMain.java src/test/resources/warnings_plugin/WarningMain2.java src/test/resources/warnings_plugin/jenkins-17787/CodeNarcReport2.xml src/test/resources/warnings_plugin/jenkins-17787/CodeNarcXmlReport.xml src/test/resources/warnings_plugin/out.txt http://jenkins-ci.org/commit/acceptance-test-harness/572a0e4dd79d457fd05a9c6a540f419711d1ac0e Log: Merge pull request #332 from stefanSchuhbaeck/warningsPlugin Warnings plugin Issue JENKINS-17787 with docker Compare: https://github.com/jenkinsci/acceptance-test-harness/compare/876baae68246...572a0e4dd79d

            drulli Ulli Hafner
            drulli Ulli Hafner
            Votes:
            1 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: