-
-
Notifications
You must be signed in to change notification settings - Fork 117
Monitor updates to pr and post Jenkins CI progress #5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
@phillipj Looking at @jbergstroem's work for the build team in that linked issue above, I see he/they are using the Github Status API. That seems like a cool way to go... maybe incorporate that in to the work you have in pollTravis? |
Yeah we should probably push to the github status API. That way, it also displays at the merge button and you don't need to find the latest comment. |
Related |
@williamkapke trying to use to status API sounds awesome! One step closer to mimic Travis completely 👯 |
moving the status api discussion to #14 |
@jbergstroem how would we know when to start polling? Could Jenkins do a HTTP requestagainst the bot, notifying there's a PR build that just started? Or should the bot listen for PR comments matching "CI: http://ci.nodejs.org/*"? |
I owe a few of you a post -- sorry for the delay. I'm a bit backlogged at the moment; will get back to you shortly. |
Seems the |
Landed with #35 |
Background and further details here: nodejs/build#236
I'd be happy to chip in here.
The text was updated successfully, but these errors were encountered: