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

Whitelist files to include when publishing #25

Merged
merged 2 commits into from
Dec 3, 2021

Conversation

MrChocolatine
Copy link
Member

@MrChocolatine MrChocolatine commented Nov 11, 2021

Build

Whitelist files to include when publishing (#25)

To reduce the final bundle size.

Before

➜ npm publish --dry-run

npm notice === Tarball Contents ===
npm notice 163B  .github/CODEOWNERS
npm notice 90B   config/environment.js
npm notice 192B  index.js
npm notice 463B  addon/helpers/reading-time.js
npm notice 80B   app/helpers/reading-time.js
npm notice 2.8kB package.json
npm notice 1.1kB LICENSE.md
npm notice 2.3kB README.md
npm notice 313B  .github/workflows/ci.yml
npm notice 375B  .github/workflows/contributors.yml
npm notice 691B  .github/dependabot.yml
npm notice 547B  .github/workflows/tag-release-publish.yml
npm notice === Tarball Details ===
npm notice name:          ember-reading-time
npm notice version:       2.0.0
npm notice package size:  3.8 kB
npm notice unpacked size: 9.1 kB
npm notice total files:   12

After

➜ npm publish --dry-run

npm notice === Tarball Contents ===
npm notice 90B   config/environment.js
npm notice 192B  index.js
npm notice 463B  addon/helpers/reading-time.js
npm notice 80B   app/helpers/reading-time.js
npm notice 2.9kB package.json
npm notice 1.1kB LICENSE.md
npm notice 2.3kB README.md
npm notice === Tarball Details ===
npm notice name:          ember-reading-time
npm notice version:       2.0.0
npm notice package size:  2.9 kB
npm notice unpacked size: 7.1 kB
npm notice total files:   7

@MrChocolatine MrChocolatine changed the title build: whitelist files to include when publishing Whitelist files to include when publishing Nov 11, 2021
@MrChocolatine MrChocolatine marked this pull request as ready for review November 11, 2021 15:23
@MrChocolatine MrChocolatine requested a review from a team as a code owner November 11, 2021 15:23
@GreatWizard GreatWizard merged commit 282b9d8 into master Dec 3, 2021
@GreatWizard GreatWizard deleted the reduce-publish-build-size branch December 3, 2021 09:35
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