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

Add support of multiple actions inside the condition

XMLWordPrintable

      It would be great to group multiple build actions by conditions.
      The current implementation causes the conditions duplication and leads to a risk of errors during theirs update.

          [JENKINS-22187] Add support of multiple actions inside the condition

          ikedam added a comment -

          https://github.com/jenkinsci/flexible-publish-plugin/pull/6

          ikedam added a comment - https://github.com/jenkinsci/flexible-publish-plugin/pull/6

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ikedam
          Path:
          src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java
          src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisher.java
          src/main/resources/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher/config.jelly
          http://jenkins-ci.org/commit/flexible-publish-plugin/a8f232f9a7fa7dcacdd7bdddc0e214231ec1871a
          Log:
          [FIXED JENKINS-22187] Supports multiple actions for a condition.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisher.java src/main/resources/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher/config.jelly http://jenkins-ci.org/commit/flexible-publish-plugin/a8f232f9a7fa7dcacdd7bdddc0e214231ec1871a Log: [FIXED JENKINS-22187] Supports multiple actions for a condition.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ikedam
          Path:
          src/test/java/org/jenkins_ci/plugins/flexible_publish/AggregationRecorder.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/ConfigurationTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/TriggerTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/AggregationRecorder.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/FileWriteBuilder.java
          src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/config.xml
          src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/jobs/migration_from_0.12/config.xml
          http://jenkins-ci.org/commit/flexible-publish-plugin/b598099d41c701977bb9273aff645fe9879a482f
          Log:
          JENKINS-22187 Added tests for multiple actions.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: src/test/java/org/jenkins_ci/plugins/flexible_publish/AggregationRecorder.java src/test/java/org/jenkins_ci/plugins/flexible_publish/ConfigurationTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/TriggerTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/AggregationRecorder.java src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/FileWriteBuilder.java src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/config.xml src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/jobs/migration_from_0.12/config.xml http://jenkins-ci.org/commit/flexible-publish-plugin/b598099d41c701977bb9273aff645fe9879a482f Log: JENKINS-22187 Added tests for multiple actions.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ikedam
          Path:
          src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java
          http://jenkins-ci.org/commit/flexible-publish-plugin/b8813b5a9032edbe5912aad23752bfb8e81f06f2
          Log:
          JENKINS-22187 wrong Action name in tests.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java http://jenkins-ci.org/commit/flexible-publish-plugin/b8813b5a9032edbe5912aad23752bfb8e81f06f2 Log: JENKINS-22187 wrong Action name in tests.

          SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: ikedam
          Path:
          src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java
          src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisher.java
          src/main/resources/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher/config.jelly
          src/test/java/org/jenkins_ci/plugins/flexible_publish/AggregationRecorder.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/ConfigurationTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/TriggerTest.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/AggregationRecorder.java
          src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/FileWriteBuilder.java
          src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/config.xml
          src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/jobs/migration_from_0.12/config.xml
          http://jenkins-ci.org/commit/flexible-publish-plugin/50f89c1d261d24831ffff17126c723e5fb785f58
          Log:
          Merge pull request #6 from ikedam/feature/JENKINS-22187_MultipleActions

          JENKINS-22187 Support multiple actions

          Compare: https://github.com/jenkinsci/flexible-publish-plugin/compare/6b9f980f351e...50f89c1d261d

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: ikedam Path: src/main/java/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher.java src/main/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisher.java src/main/resources/org/jenkins_ci/plugins/flexible_publish/ConditionalPublisher/config.jelly src/test/java/org/jenkins_ci/plugins/flexible_publish/AggregationRecorder.java src/test/java/org/jenkins_ci/plugins/flexible_publish/ConfigurationTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/MatrixAggregationTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/TriggerTest.java src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/AggregationRecorder.java src/test/java/org/jenkins_ci/plugins/flexible_publish/testutils/FileWriteBuilder.java src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/config.xml src/test/resources/org/jenkins_ci/plugins/flexible_publish/FlexiblePublisherTest/testMigrationFrom0_12/jobs/migration_from_0.12/config.xml http://jenkins-ci.org/commit/flexible-publish-plugin/50f89c1d261d24831ffff17126c723e5fb785f58 Log: Merge pull request #6 from ikedam/feature/ JENKINS-22187 _MultipleActions JENKINS-22187 Support multiple actions Compare: https://github.com/jenkinsci/flexible-publish-plugin/compare/6b9f980f351e...50f89c1d261d

          ikedam added a comment -

          Fixed in 0.13.
          It will be available in a day.
          Please try that.

          ikedam added a comment - Fixed in 0.13. It will be available in a day. Please try that.

            ikedam ikedam
            oleg_nenashev Oleg Nenashev
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: