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.

Sign up for GitHub

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

Jump to bottom

packaging: be more specific about files distributed in packed .tgz #539

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

packaging: be more specific about files distributed in packed .tgz #539

wants to merge 1 commit into from

Commits on Sep 21, 2023

  1. packaging: be more specific about files distributed in packed .tgz 

    We previously included a few unnecessary files the ui-components .tgz
file. They weren't dangerous, but weren't really appropriate for
distribution. For example:

* .parcel-cache
* .sassrc.json
* .eslint.json
* .yarnrc

Use package.json's 'files' array (an allowlist) instead of .npmignore
(a denylist) to more carefully control what's included.

Besides removing those extra files, CSS and JS source-maps are now
included in the .tgz file distributed by NPM, which should make things
easier for consumers. Since this is an open-source project, there's no
significant drawback to including those maps.
    @sjbarag
    sjbarag committed Sep 21, 2023
    Configuration menu
    Copy the full SHA
    d465afa View commit details
    Browse the repository at this point in the history