This repository has been archived by the owner on Oct 2, 2018. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 1
Remove HUGE Dependencies #159
Labels
Comments
Maybe we could remove the npm packages for testing from what's uploaded to AWS...? |
That might be hard to do... I would like to consider more aggresive options once frontend is functioning. (Like straight up killing npm/yarn) |
My proposal for now:
|
Oh actually, if we can move away from And I really doubt we need to do any css treating with sass, or whatever, we can just use the standard css... |
well, with sass we can have styles function (not styles repetition code) and some logic (loops, etc), so i thinks it is really useful and will give us a lot of freedom |
Ok, seems reasonable! |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Description
Our dependencies are huge. Like REALLY big. For context we are over three times the size of Windows 95
To Reproduce
git clean -fdX
du -hs .
(8.6M onaws-lambda
branch mostly due to .git history which we should clean up a bit maybe)lein uberjar
in backend andyarn install
in frontendlein uberjar
(still big) and 194M afteryarn install
Expected behavior
Less than 30M for everything compiled and ready. For context this is more than the size of the entire 1911 26-volume Encyclopaedia Britanicca. I'm pretty sure we can use less code than that.
Additional context
For the backend over half of the size is accounted for by the taoensso.faraday library we're using to connect to DynamoDB (again in
aws-lambda
)For the frontend, we only really want 3 things:
klud.js
? It's half a kilobyte... or maybe Canary or something)http
with Rlite for the routing?)tiny-mustache
like normal people)Microjs has a bunch of good stuff we can use.
Why does this matter?
Speed, cost of storage, speed, use of memory and speed.
Especially if we're going with an AWS pricing model were CPU and Memory Usage is the basis of our billing, so this is a way to save money. I rest my case.
The text was updated successfully, but these errors were encountered: