Skip to content
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

improve build size #76

Merged
merged 1 commit into from
Jan 28, 2022
Merged

improve build size #76

merged 1 commit into from
Jan 28, 2022

Conversation

freeman-lab
Copy link
Member

As a temporary fix to address #72 this simply replaces the LiveCode component (which was not used) with Code, and it cuts build sizes for most pages in half or more.

Before:

├ ○ /404                                   238 B           296 kB
├ ○ /blog                                  2.86 kB         298 kB
├ ○ /blog/climate-financial-risks          8.33 kB         304 kB
├ ○ /blog/climate-trace-release            18.7 kB         360 kB
├ ○ /blog/compliance-users-release         3.13 kB         299 kB
├ ○ /blog/first-post-welcome               2.57 kB         298 kB
├ ○ /blog/fsoc-open-data                   4.54 kB         300 kB
├ ○ /blog/maps-library-release             369 kB          710 kB
├ λ /blog/rss.xml                          256 B           280 kB
├ ○ /blog/soil-protocols-added             7.19 kB         303 kB
├ ○ /blog/stripe-2021-additions            4.07 kB         300 kB
├ ○ /blog/usda-csaf-comment                3.32 kB         299 kB

After:

├ ○ /404                                   236 B           135 kB
├ ○ /blog                                  2.85 kB         138 kB
├ ○ /blog/climate-financial-risks          8.32 kB         143 kB
├ ○ /blog/climate-trace-release            18.6 kB         206 kB
├ ○ /blog/compliance-users-release         3.12 kB         138 kB
├ ○ /blog/first-post-welcome               2.56 kB         138 kB
├ ○ /blog/fsoc-open-data                   4.53 kB         140 kB
├ ○ /blog/maps-library-release             369 kB          556 kB
├ λ /blog/rss.xml                          254 B           116 kB
├ ○ /blog/soil-protocols-added             7.17 kB         142 kB
├ ○ /blog/stripe-2021-additions            4.06 kB         139 kB
├ ○ /blog/usda-csaf-comment                3.31 kB         138 kB

We can consider other more involved fixes (as discussed in the issue) later, but we can do this now without losing any functionality.

@vercel
Copy link

vercel bot commented Jan 28, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/carbonplan/blog/7meSfYLJhCRNaCfTqXx6w9TMsEog
✅ Preview: https://blog-git-freeman-lab-improve-build-size-carbonplan.vercel.app

Copy link
Member

@katamartin katamartin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@freeman-lab freeman-lab merged commit 2ebccbe into main Jan 28, 2022
@freeman-lab freeman-lab deleted the freeman-lab/improve-build-size branch January 28, 2022 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants