Skip to content

Ignore some files in npm package#10882

Merged
existentialism merged 3 commits intobabel:masterfrom
JLHwung:ignore-some-files-in-npm-package
Dec 18, 2019
Merged

Ignore some files in npm package#10882
existentialism merged 3 commits intobabel:masterfrom
JLHwung:ignore-some-files-in-npm-package

Conversation

@JLHwung
Copy link
Copy Markdown
Contributor

@JLHwung JLHwung commented Dec 17, 2019

Q                       A
Fixed Issues? Some files were unexpectedly published to npm
License MIT

Ignore unused file in npm package. Here is an unpacked size comparison

package before after
generator 117 kB 116.5 kB
plugin-proposal-pipeline-operator 29.4 kB 8.3 kB
plugin-syntax-pipeline-operator 3.74 kB 3.0 kB
plugin-transform-dotall-regex 4.2 kB 2.9 kB
preset-typescript 6.34 kB 3.6 kB
standalone 3.69 MB 3.69 MB
env-standalone 4.33 MB 4.33 MB

@JLHwung JLHwung added the PR: Internal 🏠 A type of pull request used for our changelog categories label Dec 17, 2019
"files": [
"babel.js",
"babel.min.js",
"src"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I remove the src here because

  1. It is untranspiled since day 1 of @babel/standalone and nobody ever complains about that
  2. src is not intended to be used by end users

Comment thread packages/babel-plugin-syntax-pipeline-operator/.npmignore
@existentialism existentialism merged commit aa81407 into babel:master Dec 18, 2019
@github-actions github-actions Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label Mar 19, 2020
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Mar 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue PR: Internal 🏠 A type of pull request used for our changelog categories

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants