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

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

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved (View Workflow)
    • Priority: Critical
    • Resolution: Fixed
    • Component/s: warnings-plugin
    • Labels:
      None
    • Environment:
      Platform: All, OS: All
    • Similar Issues:

      Description

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

        Attachments

          Activity

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

          Show
          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.
          Hide
          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?

          Show
          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?
          Hide
          drulli Ulli Hafner added a comment -

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

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

          Seems to be missing in current version.

          Show
          docsnyder Sebastian Marsky added a comment - Seems to be missing in current version.
          Hide
          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.

          Show
          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.
          Hide
          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
          Show
          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
          Hide
          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?

          Show
          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

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

              Dates

              • Created:
                Updated:
                Resolved: