Closed
Conversation
Removed files (~47 MB working directory savings): - src/data/common-slides/ (24 MB) - presentation slides - src/data/no-solutions-transcripts/ (1.8 MB) - podcast transcripts - src/data/highlights.png (124 KB) - public/images/swag/*.png and *.jpg (22 MB) - original swag images - public/images/*.xcf (524 KB) - GIMP design source files All removed files have been archived to a separate assets repository. The swag page continues to work with WebP versions only. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Delete 7 duplicate files that exist in src/assets (boat1, boat2, map, men-wanted, funchal, smooth-sea, revealed-on-walk) - Delete 2 unused files (athens.png, ice.png) - Convert large images to WebP: tim-bouma-avatar (1.3MB→17KB), arjen-avatar (641KB→23KB), tenex-logo (1.7MB→69KB), SEC-07-smp (994KB→72KB) - Update ImageMod.astro glob pattern to include .webp files - Update references in testimonial.md, showcaseProjects.json, config.json Total reduction: 16MB → 7MB Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Resize 4 oversized images to 1920px max width - Re-compress 11 JPEGs at quality 85 - Originals preserved in assets repo Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Import image from src/assets/images instead of using ImageMod which requires file in public/images. Follows same pattern as FeatureSection.astro. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Tiered approach to reduce .git from 119MB to ~40-50MB: - Tier 2: Remove deleted directories (~25MB) - Tier 3: Remove old swag formats (~24MB) - Tier 4: Remove old image versions (~50MB) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Contributor
|
NACK, unnecessary and premature optimization. It's just us. Who cares whether it's 50MB or 100MB. The only thing that would matter is if there's deployment issues (which would probably happen once we hit 1GB+, at least that's what it was for GitHub pages, but since we deploy via Vercel it really doesn't matter). I see no benefits, only drawbacks (which you listed). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I've cleaned up unused images and optimised where necessary in #72 . All original images have been backed up in a new repo - assets (yet to be created). The website has been checked, and build passes. Now, there are several large original image copies (even as large as 5MB) stored in git history. This PR is the plan to delete them to further reduce repo bloat.
check PLAN-git-history-cleanup.md
Summary
ACK / NACK?