Rent-Cap / website

Helping Californians understand how the 2019 Renter's Protection Act affects them
https://tenantprotections.org/
6 stars 9 forks source link

Remove vips as a dependency #140

Closed wayne-li2 closed 3 years ago

wayne-li2 commented 3 years ago

vips is a pretty heavy dependency (installs 53 packages as dependencies on my Macbook).

Upon investigation, it looks like it's used only in the sharp package, which itself is only used in gatsby-plugin-manifest. This manifest is used when creating PWAs to allow users to create a shortcut image on their phone that links them to the website. This isn't really the type of behavior we need anyways (rent changes do not happen frequently enough to justify dedicating an app icon to someone's app library or homepage). Most of our users will use a mobile browser anyways.

In light of this, I removed the manifest, and dropped sharp and vips from our dependency. This will also allow us to make further improvements to our docker image (including using Alpine Linux instead of Ubuntu, a much smaller image).