Skip to content

chore: remove HISTORY.md and nonexistent CONTRIBUTING.md from tarball#388

Merged
UlisesGascon merged 1 commit intoexpressjs:masterfrom
Phillip9587:cleanup-tarball
Jan 17, 2026
Merged

chore: remove HISTORY.md and nonexistent CONTRIBUTING.md from tarball#388
UlisesGascon merged 1 commit intoexpressjs:masterfrom
Phillip9587:cleanup-tarball

Conversation

@Phillip9587
Copy link
Member

This PR removes HISTORY.md and the nonexistent 'CONTRIBUTING.md from published tarball content. It also removes automatically included files (LICENSE and README.md) from the files array.

Before:

npm pack --dry-run
npm notice
npm notice 📦  cors@2.8.5
npm notice Tarball Contents
npm notice 1.2kB HISTORY.md
npm notice 1.1kB LICENSE
npm notice 11.3kB README.md
npm notice 6.6kB lib/index.js
npm notice 999B package.json
npm notice Tarball Details
npm notice name: cors
npm notice version: 2.8.5
npm notice filename: cors-2.8.5.tgz
npm notice package size: 6.9 kB
npm notice unpacked size: 21.2 kB
npm notice shasum: 390aeb3843bc97fc0400a28c4c6e03af39f77050
npm notice integrity: sha512-0JT+RlYafxWNC[...]7O+zNNapUZRNA==
npm notice total files: 5

After:

npm pack --dry-run
npm notice
npm notice 📦  cors@2.8.5
npm notice Tarball Contents
npm notice 1.1kB LICENSE
npm notice 11.3kB README.md
npm notice 6.6kB lib/index.js
npm notice 926B package.json
npm notice Tarball Details
npm notice name: cors
npm notice version: 2.8.5
npm notice filename: cors-2.8.5.tgz
npm notice package size: 6.5 kB
npm notice unpacked size: 20.0 kB
npm notice shasum: 5a2c53a7f534cd52cb0a150c2d6fcec8a1192fd3
npm notice integrity: sha512-51Iz5vNJNv3pb[...]G7UUw/lIq8O0w==
npm notice total files: 4

@UlisesGascon UlisesGascon self-assigned this Jan 17, 2026
@UlisesGascon UlisesGascon merged commit 848e2bd into expressjs:master Jan 17, 2026
32 checks passed
@Phillip9587 Phillip9587 deleted the cleanup-tarball branch January 17, 2026 22:56
This was referenced Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants