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

Adding support for the YUI Compressor to the Hudson Warnings plug-in.

    XMLWordPrintable

Details

    • Improvement
    • Status: Resolved (View Workflow)
    • Critical
    • Resolution: Fixed
    • warnings-plugin
    • None
    • Platform: All, OS: All

    Description

      It would be nice if the Hudson Warnings plug-in support the parsing of YUI
      Compressor warnings.

      Attachments

        Activity

          Ascending order - Click to sort in descending order
          pascalra pascalra added a comment -

          Created an attachment (id=918)
          This SVN patch file (warnings-2.13-yuicompressorparser.patch) has been made based on the Hudson Warnings plug-in version "2.13". It adds the capabilities to parse the YUI Compressor warnings.

          pascalra pascalra added a comment - Created an attachment (id=918) This SVN patch file (warnings-2.13-yuicompressorparser.patch) has been made based on the Hudson Warnings plug-in version "2.13". It adds the capabilities to parse the YUI Compressor warnings.
          drulli Ulli Hafner added a comment -

          Could you please also add a small test case so that your parser will not break
          in future releases?

          drulli Ulli Hafner added a comment - Could you please also add a small test case so that your parser will not break in future releases?
          drulli Ulli Hafner added a comment -

          Thanks for the patch, code is added and will be part of the next release.

          drulli Ulli Hafner added a comment - Thanks for the patch, code is added and will be part of the next release.
          docsnyder Sebastian Marsky added a comment -

          Seems to be missing in current version.

          docsnyder Sebastian Marsky added a comment - Seems to be missing in current version.
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Ulli Hafner
          Path:
          src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          src/main/java/hudson/plugins/warnings/parser/YuiCompressorParser.java
          http://jenkins-ci.org/commit/warnings-plugin/21279d791c0fe74b6d87bba03fc74a6eb8cd1485
          Log:
          [FIXED JENKINS-4502] Added parser for YUI compressor warnings.

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ulli Hafner Path: src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java src/main/java/hudson/plugins/warnings/parser/YuiCompressorParser.java http://jenkins-ci.org/commit/warnings-plugin/21279d791c0fe74b6d87bba03fc74a6eb8cd1485 Log: [FIXED JENKINS-4502] Added parser for YUI compressor warnings.
          dogfood dogfood added a comment -

          Integrated in plugins_warnings #153
          [FIXED JENKINS-4502] Added parser for YUI compressor warnings.

          Ulli Hafner :
          Files :

          • src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java
          • src/main/java/hudson/plugins/warnings/parser/YuiCompressorParser.java
          dogfood dogfood added a comment - Integrated in plugins_warnings #153 [FIXED JENKINS-4502] Added parser for YUI compressor warnings. Ulli Hafner : Files : src/main/java/hudson/plugins/warnings/parser/ParserRegistry.java src/main/java/hudson/plugins/warnings/parser/YuiCompressorParser.java
          drulli Ulli Hafner added a comment -

          Now I see why the patch has not been applied yet. The tests are still missing...

          Can someone please attach at least a YUI output file and some assertions on how many warnings should be detected in this file?

          drulli Ulli Hafner added a comment - Now I see why the patch has not been applied yet. The tests are still missing... Can someone please attach at least a YUI output file and some assertions on how many warnings should be detected in this file?

          People

            drulli Ulli Hafner
            pascalra pascalra
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: