Skip to content
This repository has been archived by the owner on Aug 8, 2018. It is now read-only.

Add option to skip cache for static files #4

Merged
merged 5 commits into from
May 8, 2018

Conversation

esolitos
Copy link
Contributor

@esolitos esolitos commented May 3, 2018

As discussed in #3 I created this PR to allow skipping caching static files.

@csandanov
Copy link
Member

I think we should skip it by default, otherwise, there's a good chance all 64M of memory storage will be occupied by static files instead of the dynamic content. I believe that's the main issue you had in the first place.

@esolitos
Copy link
Contributor Author

esolitos commented May 7, 2018

Updated skipping cache for static files by default.

@csandanov
Copy link
Member

The build is failing for some reason, if you don't have time you can close this PR, I will review and commit these changes.

@esolitos
Copy link
Contributor Author

esolitos commented May 7, 2018

Sorry, didn't run the builkd, it was a pre-existing typo that passed unnoticed.

@csandanov csandanov merged commit 2a56b84 into wodby:master May 8, 2018
@esolitos esolitos deleted the feature/skip-static-files-caching branch May 16, 2018 10:07
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