Skip to content

Conversation

pavog
Copy link
Contributor

@pavog pavog commented May 23, 2022

Branch discovery strategy: "Only branches that also filed as PRs" excludes "master" / "main".
This pr adds a new branch discovery strategy that offers to discover branches with open PRs and/or the master/main branch.

I couldn't find an exact issue for this, but there is one for the github-branch-source-plugin.
https://issues.jenkins.io/browse/JENKINS-66102

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@pavog pavog requested a review from lafriks June 7, 2022 11:45
@pavog
Copy link
Contributor Author

pavog commented Jun 28, 2022

@lafriks ping

@lafriks lafriks merged commit 0a3b29b into jenkinsci:master Jun 28, 2022
@lafriks
Copy link
Contributor

lafriks commented Jun 28, 2022

Sorry for the delay, I was hoping for a second pair of eyes to review this but let's hope it's ok, at least from my side it looked ok

@pavog pavog deleted the pr-and-master branch June 28, 2022 07:50
@justusbunsi
Copy link
Member

Sorry for the delay, I was hoping for a second pair of eyes to review this but let's hope it's ok, at least from my side it looked ok

Sorry for not reviewing. It actually went out of my sight. My bad.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants