Skip to content

Build Optimizations - #6200

Merged
nbats merged 4 commits into
fmhy:mainfrom
bbbreaddd:opengraph-optimizations
Sep 4, 2026
Merged

Build Optimizations#6200
nbats merged 4 commits into
fmhy:mainfrom
bbbreaddd:opengraph-optimizations

Conversation

@bbbreaddd

Copy link
Copy Markdown
Contributor

Made the build time much faster
Results:

Environment Original Current Speed difference
Cloudflare Pages 2m 33s 1m 29s 1.72x
Local (i3 10100) 3m 20s 39s 5x

I'm not sure why my PC was extremely slow with the original image generation (maybe it was a fluke), but it's much improved now.

I also removed the package-lock.json and added it to gitignore so people will stop committing it. It's not supposed to be committed

@github-actions github-actions Bot added the core Core changes to our VitePress setup label Sep 2, 2026
@flowerey

flowerey commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

https://docs.npmjs.com/cli/v12/configuring-npm/package-lock-json

package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json. It describes the exact tree that was generated, such that subsequent installs are able to generate identical trees, regardless of intermediate dependency updates.

This file is intended to be committed into source repositories, and serves various purposes...

@bbbreaddd

bbbreaddd commented Sep 2, 2026

Copy link
Copy Markdown
Contributor Author

https://docs.npmjs.com/cli/v12/configuring-npm/package-lock-json

package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json. It describes the exact tree that was generated, such that subsequent installs are able to generate identical trees, regardless of intermediate dependency updates.

This file is intended to be committed into source repositories, and serves various purposes...

The main package manager for FMHY is supposed to be pnpm so we use the pnpm lock, not the package-lock.json for npm. We aren't supposed to keep multiple package lock json files for different package managers

@nbats
nbats merged commit cf647f1 into fmhy:main Sep 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core Core changes to our VitePress setup

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants