Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Internal working code #1

Closed
wants to merge 4 commits into from
Closed

Internal working code #1

wants to merge 4 commits into from

Commits on Jul 16, 2014

  1. Optimize the number of github api calls for authorization checks 

    For a user look up all their user repos & organization repos and cache for future requests.

Previously it was making an api call per jenkins job/github repo which does not scale when you have a large number of jobs.
    @alexrothenberg
    alexrothenberg committed Jul 16, 2014
    Configuration menu
    Copy the full SHA
    1c53745 View commit details
    Browse the repository at this point in the history

  2. Commented out all tests - REVERT ME SOON 

    TODO: they need to be rewritten with new mocking now that the github api calls have changed
    @alexrothenberg
    alexrothenberg committed Jul 16, 2014
    Configuration menu
    Copy the full SHA
    ea01f68 View commit details
    Browse the repository at this point in the history

  3. removed isPublicRepository check - it was not working (& not cached)

    @alexrothenberg
    alexrothenberg committed Jul 16, 2014
    Configuration menu
    Copy the full SHA
    5e2ac86 View commit details
    Browse the repository at this point in the history

Commits on Jul 29, 2014

  1. Use asList() to eagerly walk all repositories

    Matt Aken committed Jul 29, 2014
    Configuration menu
    Copy the full SHA
    27e9d2b View commit details
    Browse the repository at this point in the history