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

Avoiding excessive files in release #136

Merged
merged 1 commit into from
May 23, 2024
Merged

Avoiding excessive files in release #136

merged 1 commit into from
May 23, 2024

Conversation

RobinTail
Copy link
Contributor

@RobinTail RobinTail commented May 13, 2024 •
edited
Loading

Fixes #134

Before

$ npm pack --dry-run
npm notice
npm notice 📦  openapi3-ts@4.3.1
npm notice === Tarball Contents ===
npm notice 247B    .eslintrc
npm notice 828B    .github/workflows/ci.yaml
npm notice 159B    .idea/encodings.xml
npm notice 128B    .idea/inspectionProfiles/profiles_settings.xml
npm notice 251B    .idea/inspectionProfiles/Project_Default.xml
npm notice 187B    .idea/jsLibraryMappings.xml
npm notice 296B    .idea/markdown.xml
npm notice 274B    .idea/modules.xml
npm notice 458B    .idea/openapi3-ts.iml
npm notice 167B    .idea/vcs.xml
npm notice 117B    .prettierrc
npm notice 5.0kB   Changelog.md
npm notice 1.1kB   LICENSE
npm notice 2.0kB   README.md
npm notice 2.0kB   dist/dsl/openapi-builder30.d.ts
npm notice 2.1kB   dist/dsl/openapi-builder31.d.ts
npm notice 125B    dist/index.d.ts
npm notice 329B    dist/index.js
npm notice 89B     dist/index.js.map
npm notice 262B    dist/index.mjs
npm notice 93B     dist/index.mjs.map
npm notice 663B    dist/model/oas-common.d.ts
npm notice 9.3kB   dist/model/openapi30.d.ts
npm notice 9.5kB   dist/model/openapi31.d.ts
npm notice 796B    dist/model/server.d.ts
npm notice 499B    dist/model/specification-extension.d.ts
npm notice 4.1kB   dist/oas30-CAmzubQd.js
npm notice 23.0kB  dist/oas30-CAmzubQd.js.map
npm notice 4.6kB   dist/oas30-DyZVhJVr.mjs
npm notice 23.4kB  dist/oas30-DyZVhJVr.mjs.map
npm notice 245B    dist/oas30.d.ts
npm notice 476B    dist/oas30.js
npm notice 89B     dist/oas30.js.map
npm notice 357B    dist/oas30.mjs
npm notice 92B     dist/oas30.mjs.map
npm notice 4.8kB   dist/oas31-CiXP-okl.mjs
npm notice 24.7kB  dist/oas31-CiXP-okl.mjs.map
npm notice 4.3kB   dist/oas31-Dtl9zIuV.js
npm notice 24.3kB  dist/oas31-Dtl9zIuV.js.map
npm notice 245B    dist/oas31.d.ts
npm notice 476B    dist/oas31.js
npm notice 89B     dist/oas31.js.map
npm notice 357B    dist/oas31.mjs
npm notice 92B     dist/oas31.mjs.map
npm notice 1.2kB   dist/server-0nRY9GCE.mjs
npm notice 5.6kB   dist/server-0nRY9GCE.mjs.map
npm notice 985B    dist/server-DVBno7Ee.js
npm notice 5.5kB   dist/server-DVBno7Ee.js.map
npm notice 145B    oas30.d.ts
npm notice 145B    oas31.d.ts
npm notice 2.1kB   package.json
npm notice 14.9kB  src/dsl/openapi-builder30.spec.ts
npm notice 6.6kB   src/dsl/openapi-builder30.ts
npm notice 15.3kB  src/dsl/openapi-builder31.spec.ts
npm notice 6.9kB   src/dsl/openapi-builder31.ts
npm notice 798B    src/index.spec.ts
npm notice 124B    src/index.ts
npm notice 1.1kB   src/model/oas-common.ts
npm notice 1.8kB   src/model/openapi30.spec.ts
npm notice 12.1kB  src/model/openapi30.ts
npm notice 1.8kB   src/model/openapi31.spec.ts
npm notice 12.9kB  src/model/openapi31.ts
npm notice 868B    src/model/server.spec.ts
npm notice 1.1kB   src/model/server.ts
npm notice 1.8kB   src/model/specification-extension.spec.ts
npm notice 1.8kB   src/model/specification-extension.ts
npm notice 245B    src/oas30.ts
npm notice 245B    src/oas31.ts
npm notice 229B    test/fixture/tsconfig.node.json
npm notice 231B    test/fixture/tsconfig.node16.json
npm notice 321B    test/index.cjs
npm notice 306B    test/index.cts
npm notice 306B    test/index.mjs
npm notice 306B    test/index.mts
npm notice 4.6kB   test/node_modules/.package-lock.json
npm notice 9.9MB   test/node_modules/@esbuild/darwin-x64/bin/esbuild
npm notice 331B    test/node_modules/@esbuild/darwin-x64/package.json
npm notice 141B    test/node_modules/@esbuild/darwin-x64/README.md
npm notice 0B      test/node_modules/esbuild/bin/esbuild
npm notice 10.9kB  test/node_modules/esbuild/install.js
npm notice 21.0kB  test/node_modules/esbuild/lib/main.d.ts
npm notice 87.6kB  test/node_modules/esbuild/lib/main.js
npm notice 1.1kB   test/node_modules/esbuild/LICENSE.md
npm notice 1.3kB   test/node_modules/esbuild/package.json
npm notice 175B    test/node_modules/esbuild/README.md
npm notice 1.4kB   test/node_modules/fsevents/fsevents.d.ts
npm notice 2.5kB   test/node_modules/fsevents/fsevents.js
npm notice 163.6kB test/node_modules/fsevents/fsevents.node
npm notice 1.1kB   test/node_modules/fsevents/LICENSE
npm notice 1.3kB   test/node_modules/fsevents/package.json
npm notice 3.2kB   test/node_modules/fsevents/README.md
npm notice 17.0kB  test/node_modules/get-tsconfig/dist/index.cjs
npm notice 28.9kB  test/node_modules/get-tsconfig/dist/index.d.cts
npm notice 28.9kB  test/node_modules/get-tsconfig/dist/index.d.mts
npm notice 16.5kB  test/node_modules/get-tsconfig/dist/index.mjs
npm notice 1.1kB   test/node_modules/get-tsconfig/LICENSE
npm notice 1.0kB   test/node_modules/get-tsconfig/package.json
npm notice 7.9kB   test/node_modules/get-tsconfig/README.md
npm notice 2.1kB   test/node_modules/resolve-pkg-maps/dist/index.cjs
npm notice 497B    test/node_modules/resolve-pkg-maps/dist/index.d.cts
npm notice 497B    test/node_modules/resolve-pkg-maps/dist/index.d.mts
npm notice 2.1kB   test/node_modules/resolve-pkg-maps/dist/index.mjs
npm notice 1.1kB   test/node_modules/resolve-pkg-maps/LICENSE
npm notice 951B    test/node_modules/resolve-pkg-maps/package.json
npm notice 7.8kB   test/node_modules/resolve-pkg-maps/README.md
npm notice 2.0kB   test/node_modules/tsx/dist/cjs/index.cjs
npm notice 1.9kB   test/node_modules/tsx/dist/cjs/index.mjs
npm notice 116.9kB test/node_modules/tsx/dist/cli.cjs
npm notice 116.9kB test/node_modules/tsx/dist/cli.mjs
npm notice 364B    test/node_modules/tsx/dist/client-2f0df4a6.cjs
npm notice 330B    test/node_modules/tsx/dist/client-e665d820.mjs
npm notice 4.5kB   test/node_modules/tsx/dist/esm/index.cjs
npm notice 4.1kB   test/node_modules/tsx/dist/esm/index.mjs
npm notice 212B    test/node_modules/tsx/dist/get-pipe-path-86e97fc9.cjs
npm notice 190B    test/node_modules/tsx/dist/get-pipe-path-b74d9893.mjs
npm notice 60.1kB  test/node_modules/tsx/dist/index-5d1f01e6.cjs
npm notice 60.1kB  test/node_modules/tsx/dist/index-bd1ceb03.mjs
npm notice 1.0kB   test/node_modules/tsx/dist/loader.cjs
npm notice 995B    test/node_modules/tsx/dist/loader.mjs
npm notice 447B    test/node_modules/tsx/dist/node-features-c450ed54.cjs
npm notice 354B    test/node_modules/tsx/dist/node-features-fb266590.mjs
npm notice 30B     test/node_modules/tsx/dist/package-b2967f3e.mjs
npm notice 46B     test/node_modules/tsx/dist/package-d6a55695.cjs
npm notice 80B     test/node_modules/tsx/dist/pkgroll_create-require-b92e8e0d.mjs
npm notice 116B    test/node_modules/tsx/dist/pkgroll_create-require-c530e400.cjs
npm notice 864B    test/node_modules/tsx/dist/preflight.cjs
npm notice 826B    test/node_modules/tsx/dist/preflight.mjs
npm notice 636B    test/node_modules/tsx/dist/repl.cjs
npm notice 596B    test/node_modules/tsx/dist/repl.mjs
npm notice 344B    test/node_modules/tsx/dist/resolve-ts-path-3fca13b7.cjs
npm notice 320B    test/node_modules/tsx/dist/resolve-ts-path-eb3847f5.mjs
npm notice 321B    test/node_modules/tsx/dist/source-map.cjs
npm notice 311B    test/node_modules/tsx/dist/source-map.mjs
npm notice 497B    test/node_modules/tsx/dist/suppress-warnings.cjs
npm notice 484B    test/node_modules/tsx/dist/suppress-warnings.mjs
npm notice 156B    test/node_modules/tsx/dist/temporary-directory-2a027842.cjs
npm notice 141B    test/node_modules/tsx/dist/temporary-directory-04b36185.mjs
npm notice 1.1kB   test/node_modules/tsx/LICENSE
npm notice 1.0kB   test/node_modules/tsx/package.json
npm notice 15.5kB  test/node_modules/tsx/README.md
npm notice 573B    test/package.json
npm notice 726B    tsconfig.json
npm notice 944B    vite.config.ts
npm notice === Tarball Details ===
npm notice name:          openapi3-ts
npm notice version:       4.3.1
npm notice filename:      openapi3-ts-4.3.1.tgz
npm notice package size:  4.4 MB
npm notice unpacked size: 11.0 MB
npm notice shasum:        6568dce17035dd76eb2a1cabc9209e3341f58fc0
npm notice integrity:     sha512-TbBKorjD452Ws[...]lg0Rw/a+AO6BQ==
npm notice total files:   143

After

$ npm pack --dry-run
npm notice
npm notice 📦  openapi3-ts@4.3.1
npm notice === Tarball Contents ===
npm notice 5.0kB  Changelog.md
npm notice 1.1kB  LICENSE
npm notice 2.0kB  README.md
npm notice 2.0kB  dist/dsl/openapi-builder30.d.ts
npm notice 2.1kB  dist/dsl/openapi-builder31.d.ts
npm notice 125B   dist/index.d.ts
npm notice 329B   dist/index.js
npm notice 89B    dist/index.js.map
npm notice 262B   dist/index.mjs
npm notice 93B    dist/index.mjs.map
npm notice 663B   dist/model/oas-common.d.ts
npm notice 9.3kB  dist/model/openapi30.d.ts
npm notice 9.5kB  dist/model/openapi31.d.ts
npm notice 796B   dist/model/server.d.ts
npm notice 499B   dist/model/specification-extension.d.ts
npm notice 4.1kB  dist/oas30-CAmzubQd.js
npm notice 23.0kB dist/oas30-CAmzubQd.js.map
npm notice 4.6kB  dist/oas30-DyZVhJVr.mjs
npm notice 23.4kB dist/oas30-DyZVhJVr.mjs.map
npm notice 245B   dist/oas30.d.ts
npm notice 476B   dist/oas30.js
npm notice 89B    dist/oas30.js.map
npm notice 357B   dist/oas30.mjs
npm notice 92B    dist/oas30.mjs.map
npm notice 4.8kB  dist/oas31-CiXP-okl.mjs
npm notice 24.7kB dist/oas31-CiXP-okl.mjs.map
npm notice 4.3kB  dist/oas31-Dtl9zIuV.js
npm notice 24.3kB dist/oas31-Dtl9zIuV.js.map
npm notice 245B   dist/oas31.d.ts
npm notice 476B   dist/oas31.js
npm notice 89B    dist/oas31.js.map
npm notice 357B   dist/oas31.mjs
npm notice 92B    dist/oas31.mjs.map
npm notice 1.2kB  dist/server-0nRY9GCE.mjs
npm notice 5.6kB  dist/server-0nRY9GCE.mjs.map
npm notice 985B   dist/server-DVBno7Ee.js
npm notice 5.5kB  dist/server-DVBno7Ee.js.map
npm notice 2.2kB  package.json
npm notice === Tarball Details ===
npm notice name:          openapi3-ts
npm notice version:       4.3.1
npm notice filename:      openapi3-ts-4.3.1.tgz
npm notice package size:  20.0 kB
npm notice unpacked size: 165.0 kB
npm notice shasum:        e3e40636e307573d7f75d9ff79480d8880928667
npm notice integrity:     sha512-wjwqVZ0tvBMtN[...]WJ5muxg21x/qQ==
npm notice total files:   38

@RobinTail RobinTail marked this pull request as ready for review May 13, 2024 19:40
@RobinTail
Copy link
Contributor Author

@pjmolina , please review.

@RobinTail RobinTail mentioned this pull request May 13, 2024
Closed
RobinTail
RobinTail commented May 13, 2024
View reviewed changes
package.json
@@ -64,5 +64,6 @@
"vite": "^5.2.7",
"vitest": "^1.4.0",
"vitest-teamcity-reporter": "^0.3.0"
}
},
"files": ["dist", "*.md"]
Copy link
Contributor Author

@RobinTail RobinTail May 13, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

LICENSE and package.json are always included

https://docs.npmjs.com/cli/v10/configuring-npm/package-json#files

@RobinTail
Copy link
Contributor Author

@pjmolina , please review

@pjmolina pjmolina merged commit 1165ddd into metadevpro:master May 23, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Source files are published in the package
2 participants
@RobinTail @pjmolina