Skip to content
This repository has been archived by the owner on Nov 21, 2022. It is now read-only.

Reduced risk of reaching rate limit on github API #7

Merged
merged 3 commits into from
Oct 17, 2015
Merged

Reduced risk of reaching rate limit on github API #7

merged 3 commits into from
Oct 17, 2015

Conversation

fdocr
Copy link
Contributor

@fdocr fdocr commented Oct 16, 2015

Implemented with memory-cache

"If-None-Match": cache.get('ETag?' + username + '&' + pageNumber)
};
}
console.log(options);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Remove debug.

@jenkoian
Copy link
Owner

Dude this is fantastic, exactly what I had in mind, few minor things but looks great :)

@fdocr
Copy link
Contributor Author

fdocr commented Oct 17, 2015

Awesome, it's actually my first PR, learned a thing or two in the process... Now it's time to get that counter running :P

jenkoian added a commit that referenced this pull request Oct 17, 2015
Reduced risk of reaching rate limit on github API
@jenkoian jenkoian merged commit 2b77720 into jenkoian:master Oct 17, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants