-
Type:
Bug
-
Status: Resolved (View Workflow)
-
Priority:
Minor
-
Resolution: Fixed
-
Component/s: github-branch-source-plugin
-
Labels:None
-
Similar Issues:
the regex "^PR-(
d+)(.)?$" includes {} in the second matching group which means PR-11-HEAD will never match the strategy of HEAD as the second matching group includes the {}. Just need to do something like "^PR(
d+)(:?-(.))?$" (if I can remember my regex for non-matching groups correctly)
- mentioned in
-
Page Loading...
Field | Original Value | New Value |
---|---|---|
Status | Open [ 1 ] | Resolved [ 5 ] |
Resolution | Fixed [ 1 ] |
Remote Link | This issue links to "Page (Jenkins Wiki)" [ 20259 ] |
Code changed in jenkins
User: Stephen Connolly
Path:
src/main/java/org/jenkinsci/plugins/github_branch_source/GitHubSCMSource.java
http://jenkins-ci.org/commit/github-branch-source-plugin/8293288381f909034e1f754da69fe57afdb5cd94
Log:
Merge pull request #178 from stephenc/more-logging
[FIXED JENKINS-49945] Fix regex and add some logging to help diagnose issues in resolve(thingName,listener)
Compare: https://github.com/jenkinsci/github-branch-source-plugin/compare/d60cc7617ee9...8293288381f9