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

Improve current behavior of description concatenation

      This effort is follow-up of JENKINS-26671 and can solve JENKINS-28006 at least partially.

      Current behavior is a new description is always concatenated with the previous one. In addition, description from build phase(s) is mixed with the post-build description as well.

      I'm going to change that behavior and leave the user to decide when description should be added like a concatenation (current state) and when the description should rewrite it.

          [JENKINS-30167] Improve current behavior of description concatenation

          PR sent.

          Pavel Janoušek added a comment - PR sent.

          Code changed in jenkins
          User: Ing. Pavel Janousek
          Path:
          .gitignore
          pom.xml
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilder.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/config.jelly
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/config.jelly
          src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilderTest.java
          src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisherTest.java
          http://jenkins-ci.org/commit/description-setter-plugin/1910cb65da8583561cf209c96155c18dbc0eace7
          Log:
          JENKINS-30167 Added an option for driving a concatenation mode

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Ing. Pavel Janousek Path: .gitignore pom.xml src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilder.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/config.jelly src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/config.jelly src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilderTest.java src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisherTest.java http://jenkins-ci.org/commit/description-setter-plugin/1910cb65da8583561cf209c96155c18dbc0eace7 Log: JENKINS-30167 Added an option for driving a concatenation mode

          Code changed in jenkins
          User: Pavel Janousek
          Path:
          .gitignore
          pom.xml
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilder.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java
          src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/config.jelly
          src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/config.jelly
          src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilderTest.java
          src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisherTest.java
          http://jenkins-ci.org/commit/description-setter-plugin/da0d0e96ef7a26e0685a9a31f688b718ba50a2b2
          Log:
          Merge pull request #9 from pjanouse/JENKINS-30167

          JENKINS-30167 Added an option for driving a concatenation mode

          Compare: https://github.com/jenkinsci/description-setter-plugin/compare/2efd75864daa...da0d0e96ef7a

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Pavel Janousek Path: .gitignore pom.xml src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilder.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterHelper.java src/main/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisher.java src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterBuilder/config.jelly src/main/resources/hudson/plugins/descriptionsetter/DescriptionSetterPublisher/config.jelly src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterBuilderTest.java src/test/java/hudson/plugins/descriptionsetter/DescriptionSetterPublisherTest.java http://jenkins-ci.org/commit/description-setter-plugin/da0d0e96ef7a26e0685a9a31f688b718ba50a2b2 Log: Merge pull request #9 from pjanouse/ JENKINS-30167 JENKINS-30167 Added an option for driving a concatenation mode Compare: https://github.com/jenkinsci/description-setter-plugin/compare/2efd75864daa...da0d0e96ef7a

          Merged...

          Pavel Janoušek added a comment - Merged...

          Filipe Silva added a comment -

          Please release this. I really need it urgently!

          Filipe Silva added a comment - Please release this. I really need it urgently!

          fsilva I'm not a maintainer of this plug-in you should ask the owner.

          Pavel Janoušek added a comment - fsilva I'm not a maintainer of this plug-in you should ask the owner.

          As plugin is still not released (2 years since last release), I'll share already built snapshot including this fix. I also needed this functionality.
          description-setter-plugin-1.11-SNAPSHOT.tar.gz
          description-setter-plugin-1.11-SNAPSHOT.zip

          Maciej Załucki added a comment - As plugin is still not released (2 years since last release), I'll share already built snapshot including this fix. I also needed this functionality. description-setter-plugin-1.11-SNAPSHOT.tar.gz description-setter-plugin-1.11-SNAPSHOT.zip

          I found out that this improvement was not applied to matrix build description. The best idea would be adding second checkbox if we want to concatenate it fox matrix build or not. Anyways, I've done small fix so current setting is also applied to matrix build description.
          Code changes
          description-setter-plugin-1.11.2-SNAPSHOT.zip
          description-setter-plugin-1.11.2-SNAPSHOT.tar.gz

          Maciej Załucki added a comment - I found out that this improvement was not applied to matrix build description. The best idea would be adding second checkbox if we want to concatenate it fox matrix build or not. Anyways, I've done small fix so current setting is also applied to matrix build description. Code changes description-setter-plugin-1.11.2-SNAPSHOT.zip description-setter-plugin-1.11.2-SNAPSHOT.tar.gz

            pajasoft Pavel Janoušek
            pajasoft Pavel Janoušek
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

              Created:
              Updated:
              Resolved: