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 #172

Merged
merged 2 commits into from
Jan 17, 2022

Conversation

MrChocolatine
Copy link
Member

@MrChocolatine MrChocolatine commented Nov 11, 2021

Build

Whitelist files to include when publishing (#172)

To reduce the final bundle size.

Before

➜ npm publish --dry-run

npm notice === Tarball Contents ===
npm notice 22B    .github/CODEOWNERS
npm notice 1.6kB  addon/components/feature-controls.hbs
npm notice 191B   addon/templates/features-list.hbs
npm notice 87B    config/environment.js
npm notice 202B   addon/services/feature-control-storage.js
npm notice 82B    app/services/feature-control-storage.js
npm notice 3.1kB  addon/components/feature-controls.js
npm notice 77B    app/components/feature-controls.js
npm notice 125B   app/storages/feature-controls.js
npm notice 74B    addon/routes/features-list.js
npm notice 70B    app/routes/features-list.js
npm notice 73B    app/templates/features-list.js
npm notice 70B    index.js
npm notice 794B   addon/instance-initializers/load-feature-controls.js
npm notice 110B   app/instance-initializers/load-feature-controls.js
npm notice 110B   addon/route-setup.js
npm notice 66B    addon/utils/window.js
npm notice 3.0kB  package.json
npm notice 1.1kB  LICENSE.md
npm notice 6.5kB  README.md
npm notice 84.5kB docs/screenshot.png
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.yml
npm notice === Tarball Details ===
npm notice name:          ember-feature-controls
npm notice version:       3.0.0
npm notice package size:  74.9 kB
npm notice unpacked size: 103.8 kB
npm notice total files:   25

After

➜ npm publish --dry-run

npm notice === Tarball Contents ===
npm notice 1.6kB addon/components/feature-controls.hbs
npm notice 191B  addon/templates/features-list.hbs
npm notice 87B   config/environment.js
npm notice 202B  addon/services/feature-control-storage.js
npm notice 82B   app/services/feature-control-storage.js
npm notice 3.1kB addon/components/feature-controls.js
npm notice 77B   app/components/feature-controls.js
npm notice 125B  app/storages/feature-controls.js
npm notice 74B   addon/routes/features-list.js
npm notice 70B   app/routes/features-list.js
npm notice 73B   app/templates/features-list.js
npm notice 70B   index.js
npm notice 794B  addon/instance-initializers/load-feature-controls.js
npm notice 110B  app/instance-initializers/load-feature-controls.js
npm notice 110B  addon/route-setup.js
npm notice 66B   addon/utils/window.js
npm notice 3.0kB package.json
npm notice 1.1kB LICENSE.md
npm notice 6.5kB README.md
npm notice === Tarball Details ===
npm notice name:          ember-feature-controls
npm notice version:       3.0.0
npm notice package size:  6.2 kB
npm notice unpacked size: 17.5 kB
npm notice total files:   19

@MrChocolatine MrChocolatine changed the title reduce publish build size Whitelist files to include on publish Nov 11, 2021
@MrChocolatine MrChocolatine changed the title Whitelist files to include on publish Whitelist files to include when publishing Nov 11, 2021
@MrChocolatine MrChocolatine marked this pull request as ready for review November 11, 2021 15:22
@MrChocolatine MrChocolatine requested a review from a team as a code owner November 11, 2021 15:22
@GreatWizard GreatWizard merged commit bfdf316 into master Jan 17, 2022
@GreatWizard GreatWizard deleted the reduce-publish-build-size branch January 17, 2022 14:39
@GreatWizard GreatWizard mentioned this pull request Jan 21, 2022
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