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

Migrate to tabs #49

Merged
merged 7 commits into from
Feb 18, 2024
Merged

Migrate to tabs #49

merged 7 commits into from
Feb 18, 2024

Conversation

antonk52
Copy link
Owner

@antonk52 antonk52 commented Feb 18, 2024

No runtime changes

Since lilconfig's one of the core feature is being small in size and it is no longer uses typescript to transpile. It means that we now fully control the javascript that we ship to npm. Migrating to tabs instead of spaces saves some space for the onyl runtime file shipped. Specifically it went from 15.2kB to 11.4kB ~ 25% smaller. Also updated package.json to use 2 spaces which is the npm default.

Before

▲ npm pack --dry-run
npm notice
npm notice 📦  [email protected]
npm notice === Tarball Contents ===
npm notice 1.1kB  LICENSE
npm notice 1.2kB  package.json
npm notice 2.6kB  readme.md
npm notice 1.7kB  src/index.d.ts
npm notice 15.2kB src/index.js

After

▲ npm pack --dry-run
npm notice
npm notice 📦  [email protected]
npm notice === Tarball Contents ===
npm notice 1.1kB  LICENSE
npm notice 1.1kB  package.json
npm notice 2.6kB  readme.md
npm notice 1.6kB  src/index.d.ts
npm notice 11.4kB src/index.js

@antonk52 antonk52 merged commit 5b986e1 into master Feb 18, 2024
7 of 9 checks passed
@antonk52 antonk52 deleted the feat/move-stuff-around branch February 18, 2024 13:24
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