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

limit is unnecessary; plus, servers are capable of compressing resour… #46

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

aliakseiherman
Copy link

@aliakseiherman aliakseiherman commented Sep 4, 2022

…ces;

otherwise, 32768 should be increased at least to 500KB — some fonts are far beyond 200 KB in size;
one more reason for deleting the Default Limit — the code doesn't throw exception — you only get message in console logs and it's easy to overlook the details in output — which means Important Data can be ignored by the script and this leads to errors in production;

…ces;

otherwise, 32768 should be increased at least to 500KB — some fonts are far beyond 200 KB in size;
one more reason for deleting the Default Limit — it doesn't throw exception and break — you only get message in console logs and it's easy to overlook the details in output — which means Important Data can be ignored by the script and this leads to errors in production;
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.

None yet

1 participant