Skip to content

Cache requests between subsequent calls#22

Merged
joehoyle merged 7 commits intomasterfrom
cache-requests
Oct 3, 2019
Merged

Cache requests between subsequent calls#22
joehoyle merged 7 commits intomasterfrom
cache-requests

Conversation

@rmccue
Copy link
Copy Markdown
Member

@rmccue rmccue commented Aug 23, 2018

This avoids calling out to the API multiple times in a row when not needed.

Fixes #18.

This avoids calling out to the API multiple times in a row when not
needed.
Copy link
Copy Markdown

@hm-linter hm-linter Bot left a comment

Choose a reason for hiding this comment

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

Linting failed (122 errors).

(119 notices occurred in your codebase, but were on files/lines not included in this PR.)

Copy link
Copy Markdown
Member

@joehoyle joehoyle left a comment

Choose a reason for hiding this comment

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

Looks and works well from what I can see

@joehoyle joehoyle merged commit 1f3f6f1 into master Oct 3, 2019
@joehoyle joehoyle deleted the cache-requests branch October 3, 2019 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use shared request promise pool

2 participants