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

Remove support for caching #54

Closed
wants to merge 2 commits into from
Closed

Remove support for caching #54

wants to merge 2 commits into from

Conversation

Atul9
Copy link

@Atul9 Atul9 commented Mar 7, 2018

#46

@iffyuva
Copy link
Member

iffyuva commented Mar 9, 2018

@Atul9 PR looks good, can you check why specs are failing?

@Atul9 Atul9 requested a review from iffyuva March 9, 2018 04:12
@Atul9 Atul9 force-pushed the remove-caching-support branch from caca05d to 52639b8 Compare April 3, 2018 16:02
@iffyuva
Copy link
Member

iffyuva commented Oct 9, 2020

@Atul9 this needs to be re-thought. I'm closing this PR for now, but will keep an eye for cache optimizations based on this PR.

@iffyuva iffyuva closed this Oct 9, 2020
@iffyuva iffyuva deleted the remove-caching-support branch October 9, 2020 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@iffyuva iffyuva Awaiting requested review from iffyuva

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Atul9 @iffyuva