8000 Adjustments by jglick · Pull Request #2 · jenkinsci/github-branch-source-plugin · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@jglick
Copy link
Member
@jglick jglick commented Oct 8, 2015

@ghost
Copy link
ghost commented Oct 8, 2015

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

@amuniz
Copy link
Member
amuniz commented Oct 9, 2015

🐝
BTW How is this compiling? Did you deploy snapshots for upstreams?

@tfennelly
Copy link
Member

🐝

Though the discoverSources/ visitSources change seems to be bin-incompatible.

@jglick
Copy link
Member Author
jglick commented Oct 9, 2015

Did you deploy snapshots for upstreams?

Yes, I did start.

the discoverSources/ visitSources change seems to be bin-incompatible.

Yes of course, but there has never been a release of the API to begin with.

@tfennelly
Copy link
Member

Yes of course, but there has never been a release of the API to begin with.

Fair enough. Thanks and sorry for the noise.

@stephenc
Copy link
Member

🐝

jglick added a commit that referenced this pull request Oct 20, 2015
@jglick jglick merged commit cfb2e8b into jenkinsci:master Oct 20, 2015
@jglick jglick deleted the adjustments branch October 20, 2015 12:08
dubrsl pushed a commit to dubrsl/github-branch-source-plugin that referenced this pull request Oct 17, 2020
dubrsl pushed a commit to dubrsl/github-branch-source-plugin that referenced this pull request Oct 17, 2020
[JENKINS-58361, 57760] Feature/web hook
@jtnord jtnord mentioned this pull request Feb 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

0