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

Replace Codenarc parser with native parser

    XMLWordPrintable

Details

    Description

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

      Attachments

        Issue Links

          Activity

            Is there any update?

            pedrofurlanetto Pedro Furlanetto added a comment - Is there any update?
            drulli Ulli Hafner added a comment -

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

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

            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 )

            pedrofurlanetto 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 )
            drulli 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.

            drulli 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.

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

            pedrofurlanetto 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
            drulli Ulli Hafner added a comment -

            Fine, thanks!

            drulli Ulli Hafner added a comment - Fine, thanks!

            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_issue_link 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

            People

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

              Dates

                Created:
                Updated:
                Resolved: