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

Add support of multiple actions inside the condition

    XMLWordPrintable

Details

    Description

      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.

      Attachments

        Issue Links

          Activity

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

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

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

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

            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_issue_link 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 ikedam added a comment -

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

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

            People

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

              Dates

                Created:
                Updated:
                Resolved: