-
Notifications
You must be signed in to change notification settings - Fork 30
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
Decrease the app size #5
Comments
Yeah agreed, that is way too large. I'd more than welcome a PR if you are able to! Alternatively, I'll check into where I can cut the app size as soon as I can |
Hi, I can take a look at this |
All yours @dhuang612 - be great to have another contribution! 👍 |
created PR #112 |
I wasn't involved in this when #112 was closed so I'll defer to you @joshghent ; can we consider this closed now? |
@jamescallumyoung Well let's check the app size now and see if it's acceptable. Maybe this can push forward the Github Actions CI work so we can measure the app size? |
Sounds good. Lets leave it for now then and see what the current size is. |
First of all, thank you for open-sourcing this, it's a really cool app!
I think there should be a small research on why the app size is
279.7 MB
, I believe it's possible to decrease that drastically as it's just a menubar app, I noticed that it's using packages likelodash
which is pretty big but as it's usingparcel
(which I don't have much experience on) I'm not sure if it's doing properly tree-shaking or not.The text was updated successfully, but these errors were encountered: