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

expose leader board via api #9

Merged
merged 2 commits into from Aug 20, 2015

Conversation

Projects
None yet
7 participants
@iczechowski
Contributor

iczechowski commented May 13, 2013

Hi,

I would like to have access to ci-game leader board via API, so that standings may be read and consumed with external application like dashboard.

Change seams to be trivial.

Please, review and comment and don't hesitate to tell if anything is missing in the pull request.

Cheers,
Igor

@buildhive

This comment has been minimized.

Show comment
Hide comment
@buildhive

buildhive May 13, 2013

Jenkins » ci-game-plugin #12 FAILURE
Looks like there's a problem with this pull request
(what's this?)

buildhive commented May 13, 2013

Jenkins » ci-game-plugin #12 FAILURE
Looks like there's a problem with this pull request
(what's this?)

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost May 19, 2013

+1 for use on Geckoboard.

ghost commented May 19, 2013

+1 for use on Geckoboard.

@jenkinsadmin

This comment has been minimized.

Show comment
Hide comment
@jenkinsadmin

jenkinsadmin Jul 17, 2013

Member

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

Member

jenkinsadmin commented Jul 17, 2013

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

@jdonaldson10

This comment has been minimized.

Show comment
Hide comment
@jdonaldson10

jdonaldson10 Dec 7, 2013

+1 would definitely use this

jdonaldson10 commented Dec 7, 2013

+1 would definitely use this

@bbankowski

This comment has been minimized.

Show comment
Hide comment
@bbankowski

bbankowski Jul 30, 2015

Is it going to be added to anytime soon? We use it with dashing, it is a really neat feature.

bbankowski commented Jul 30, 2015

Is it going to be added to anytime soon? We use it with dashing, it is a really neat feature.

@simschla

This comment has been minimized.

Show comment
Hide comment
@simschla

simschla Aug 20, 2015

Member

I see no reason why this should not be included!

Member

simschla commented Aug 20, 2015

I see no reason why this should not be included!

simschla added a commit that referenced this pull request Aug 20, 2015

Merge pull request #9 from iczechowski/master
expose leader board via api

@simschla simschla merged commit 484854e into jenkinsci:master Aug 20, 2015

1 check passed

default This pull request looks good
Details
@iczechowski

This comment has been minimized.

Show comment
Hide comment
@iczechowski

iczechowski Aug 20, 2015

Contributor

thanks, 👍
bye bye private fork.

Contributor

iczechowski commented Aug 20, 2015

thanks, 👍
bye bye private fork.

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