-
Notifications
You must be signed in to change notification settings - Fork 622
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.
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
Some libraries have outgrown the limit... (and some don't fit) #818
Comments
Hi again @isiahmeadows ! I stumble upon you in many places 😄 Agree with you about oversized libs in the list! The 5K limit is weird anyway.... The reason we want these libraries to be small is so they will load fast. This would make either 4K or 8K a better number as it aligns better with network packet sizes, disk sectors etc. 8K does seem big for a micro-framework though... |
Perhaps a solution could be to divide the libraries into categories like these:
That way it wouldn't be necessary to remove what exceeds 5 KB (just recategorize it) and that would also encourage more people to submit commits of libraries that meet the other requirements except for the 5 KB limit. |
Specifically these, as per the website (sizes as of March 21):
Out of these, some I know do more than one thing:
Maybe, some of these should be commented out/removed?
The text was updated successfully, but these errors were encountered: