Skip to content

Reduce bundle size of this package, fix CI#494

Open
tbantle22 wants to merge 5 commits intoliuliu-dev:mainfrom
dolthub:taylor/bundle
Open

Reduce bundle size of this package, fix CI#494
tbantle22 wants to merge 5 commits intoliuliu-dev:mainfrom
dolthub:taylor/bundle

Conversation

@tbantle22
Copy link

  ┌─────────────┬─────────┬────────┬───────────┐
  │ Metric      │ Before  │ After  │ Reduction │
  ├─────────────┼─────────┼────────┼───────────┤
  │ ESM raw     │ 12.0 MB │ 92 KB  │ 99.2%     │
  │ CJS raw     │ 12.0 MB │ 93 KB  │ 99.2%     │
  │ ESM gzipped │ 2.2 MB  │ 2.6 KB │ 99.8%     │
  └─────────────┴─────────┴────────┴───────────┘

For reducing bundle size:

  • Removes large dependencies that are easy to implement inline
  • Remove unused dependencies and code

For CI:

  • Github actions workflow file used the wrong path so it wasn't running on PRs
  • Removed next (not really necessary since it doesn't use pages)
  • Get eslint and tests running and passing

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.

1 participant