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

Fix various rate limiting issues #1424

Merged
merged 10 commits into from
Jul 31, 2020
Merged

Fix various rate limiting issues #1424

merged 10 commits into from
Jul 31, 2020

Commits on Jul 30, 2020

  1. retry after abuse limit

    @hipstersmoothie
    hipstersmoothie committed Jul 30, 2020
    Configuration menu
    Copy the full SHA
    80b3922 View commit details
    Browse the repository at this point in the history

  2. memoize searchRepo

    @hipstersmoothie
    hipstersmoothie committed Jul 30, 2020
    Configuration menu
    Copy the full SHA
    84109da View commit details
    Browse the repository at this point in the history

  3. pretty print time left on rate limit 

    better time remaining messaging
    @hipstersmoothie
    hipstersmoothie committed Jul 30, 2020
    Configuration menu
    Copy the full SHA
    0ef0b9a View commit details
    Browse the repository at this point in the history

Commits on Jul 31, 2020

  1. use graphql api on octokit instance

    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    3d64d66 View commit details
    Browse the repository at this point in the history

  2. mock octokit in cli tests

    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    b4831c0 View commit details
    Browse the repository at this point in the history

  3. return only commits in the canary release instead of all commits sinc… 

    …e last tag. getting all commits can trigger rate limits and canaries should be lightweight
    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    66f7e2a View commit details
    Browse the repository at this point in the history

  4. remove author email match, easily hits rate limit

    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    5d9bb97 View commit details
    Browse the repository at this point in the history

  5. memoize getCommits

    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    2a0f270 View commit details
    Browse the repository at this point in the history

  6. call getPullRequest 1 less time

    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    baab296 View commit details
    Browse the repository at this point in the history

  7. search for past contributions serially so that memoized graphql call … 

    …works
    @hipstersmoothie
    hipstersmoothie committed Jul 31, 2020
    Configuration menu
    Copy the full SHA
    d6e7be2 View commit details
    Browse the repository at this point in the history