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

GitLabTriggerPlugin Ignores Regex Match in DSL

    Details

    • Similar Issues:

      Description

      There is a flaw in the DSL support for the gitlab-plugin which causes the trigger to only work for standard matching, not regex. This is the default for the gitlab-plugin, so it just appears to work if you're not trying to use the regex match. I have a patch already written for this and will be submitting a pull request momentarily.

        Attachments

          Activity

          Hide
          daspilker Daniel Spilker added a comment -

          Will be fixed in 1.58.

          Show
          daspilker Daniel Spilker added a comment - Will be fixed in 1.58.
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Daniel Spilker
          Path:
          docs/Home.md
          job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/triggers/GitLabTriggerContext.groovy
          job-dsl-core/src/test/groovy/javaposse/jobdsl/dsl/helpers/triggers/TriggerContextSpec.groovy
          http://jenkins-ci.org/commit/job-dsl-plugin/b28e859123c0a6acc6c745ea35a46f2af26405b2
          Log:
          Merge remote-tracking branch 'isugimpy/JENKINS-41789'

          Compare: https://github.com/jenkinsci/job-dsl-plugin/compare/9fdd8567f838...b28e859123c0

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Daniel Spilker Path: docs/Home.md job-dsl-core/src/main/groovy/javaposse/jobdsl/dsl/helpers/triggers/GitLabTriggerContext.groovy job-dsl-core/src/test/groovy/javaposse/jobdsl/dsl/helpers/triggers/TriggerContextSpec.groovy http://jenkins-ci.org/commit/job-dsl-plugin/b28e859123c0a6acc6c745ea35a46f2af26405b2 Log: Merge remote-tracking branch 'isugimpy/ JENKINS-41789 ' Compare: https://github.com/jenkinsci/job-dsl-plugin/compare/9fdd8567f838...b28e859123c0
          Hide
          isugimpy Jayme Howard added a comment -
          Show
          isugimpy Jayme Howard added a comment - https://github.com/jenkinsci/job-dsl-plugin/pull/992 should be all set to merge.

            People

            • Assignee:
              daspilker Daniel Spilker
              Reporter:
              isugimpy Jayme Howard
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: