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

Cache fingerprint #26

Closed
stryju opened this issue Jun 23, 2015 · 5 comments
Closed

Cache fingerprint #26

stryju opened this issue Jun 23, 2015 · 5 comments
Labels
enhancement
Milestone
modernize

Comments

@stryju
Copy link

stryju commented Jun 23, 2015

wouldn't it make sense to store fingerprint in a variable and only run it once when needed?
@ericelliott
Copy link
Collaborator

Sure, but we should modularize the build first.

@ericelliott ericelliott modified the milestones: 1.x, modernize Jun 23, 2015
@ericelliott ericelliott changed the title fingerprinting Cache fingerprint Jul 5, 2015
@ericelliott
Copy link
Collaborator

The build is better now. Would you like to take a crack at caching the fingerprint?

@stryju
Copy link
Author

stryju commented Aug 21, 2015

this seems to be done already
https://github.com/ericelliott/cuid/blob/master/source/client/index.js#L4
https://github.com/ericelliott/cuid/blob/master/source/server/index.js#L4

@ericelliott
Copy link
Collaborator

Cool! =)

@stryju
Copy link
Author

stryju commented Aug 24, 2015

caching-related issue(s) moved to the appropriate repo(s)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
modernize
Development

No branches or pull requests
2 participants
@stryju @ericelliott