Skip to content

package.json: explicitly set the files to install.#285

Merged
jonaskello merged 2 commits intojonaskello:masterfrom
XhmikosR:xmr/package-json-files
Feb 23, 2026
Merged

package.json: explicitly set the files to install.#285
jonaskello merged 2 commits intojonaskello:masterfrom
XhmikosR:xmr/package-json-files

Conversation

@XhmikosR
Copy link
Copy Markdown
Contributor

No need for an extra .npmignore file

C:\Users\xmr\Desktop\tsconfig-paths>git co master
Switched to branch 'master'
Your branch is up to date with 'origin/master'.

C:\Users\xmr\Desktop\tsconfig-paths>npm pack --dry-run
npm notice
npm notice package: tsconfig-paths@4.2.0
npm notice Tarball Contents
npm notice 14.1kB CHANGELOG.md
npm notice 1.1kB LICENSE
npm notice 10.3kB README.md
npm notice 2.2kB package.json
npm notice 27B register.js
npm notice 2.4kB src/config-loader.ts
npm notice 2.5kB src/filesystem.ts
npm notice 546B src/index.ts
npm notice 2.1kB src/mapping-entry.ts
npm notice 6.2kB src/match-path-async.ts
npm notice 4.8kB src/match-path-sync.ts
npm notice 3.4kB src/register.ts
npm notice 3.8kB src/try-path.ts
npm notice 6.3kB src/tsconfig-loader.ts
npm notice Tarball Details
npm notice name: tsconfig-paths
npm notice version: 4.2.0
npm notice filename: tsconfig-paths-4.2.0.tgz
npm notice package size: 15.2 kB
npm notice unpacked size: 59.6 kB
npm notice shasum: 8b34c4d6b3afd372bfdac733e6a52862cd692d0d
npm notice integrity: sha512-rMd8/xt3e2qHf[...]9SZv5rN/WzqUg==
npm notice total files: 14
npm notice
tsconfig-paths-4.2.0.tgz

C:\Users\xmr\Desktop\tsconfig-paths>git co -
Switched to branch 'xmr/package-json-files'

C:\Users\xmr\Desktop\tsconfig-paths>npm pack --dry-run
npm notice
npm notice package: tsconfig-paths@4.2.0
npm notice Tarball Contents
npm notice 14.1kB CHANGELOG.md
npm notice 1.1kB LICENSE
npm notice 10.3kB README.md
npm notice 2.4kB package.json
npm notice 27B register.js
npm notice 2.4kB src/config-loader.ts
npm notice 2.5kB src/filesystem.ts
npm notice 546B src/index.ts
npm notice 2.1kB src/mapping-entry.ts
npm notice 6.2kB src/match-path-async.ts
npm notice 4.8kB src/match-path-sync.ts
npm notice 3.4kB src/register.ts
npm notice 3.8kB src/try-path.ts
npm notice 6.3kB src/tsconfig-loader.ts
npm notice Tarball Details
npm notice name: tsconfig-paths
npm notice version: 4.2.0
npm notice filename: tsconfig-paths-4.2.0.tgz
npm notice package size: 15.2 kB
npm notice unpacked size: 59.8 kB
npm notice shasum: 6e1ef42dcd137a1dd72d2bc77091946a483814d4
npm notice integrity: sha512-GudiDDSkX6EKA[...]SaOdh2J22Hl5w==
npm notice total files: 14
npm notice
tsconfig-paths-4.2.0.tgz

PR #250 changed this, but IMHO it was unnecessary. The same result can be achieved with excludes.

Also, changed the repository URL to the canonical form with npm pkg fix.

/CC @jonaskello: minor change, I just came across this because the currently published v4.2.0 does include the test files: https://www.npmjs.com/package/tsconfig-paths/v/4.2.0?activeTab=code

@XhmikosR XhmikosR force-pushed the xmr/package-json-files branch from 6e9cb65 to df4e55b Compare February 22, 2026 06:53
@jonaskello
Copy link
Copy Markdown
Owner

@XhmikosR Could you move the repo urls change commit to a separate PR? That would make it easier for me to track the changes.

@XhmikosR XhmikosR force-pushed the xmr/package-json-files branch from df4e55b to 9162229 Compare February 23, 2026 05:10
@XhmikosR
Copy link
Copy Markdown
Contributor Author

@jonaskello: done

@jonaskello
Copy link
Copy Markdown
Owner

Great, this PR is now only the package.json updates.
Did you make a separate PR the url changes? I could not find it.

@XhmikosR
Copy link
Copy Markdown
Contributor Author

XhmikosR commented Feb 23, 2026 via email

@XhmikosR XhmikosR force-pushed the xmr/package-json-files branch from 6475af4 to a5ca203 Compare February 23, 2026 07:00
No need for an extra .npmignore file
@XhmikosR XhmikosR force-pushed the xmr/package-json-files branch from a5ca203 to 0a4426c Compare February 23, 2026 07:03
Comment thread package.json Outdated
@XhmikosR
Copy link
Copy Markdown
Contributor Author

Should be good to go now @jonaskello 👍

@XhmikosR
Copy link
Copy Markdown
Contributor Author

BTW if you plan to cut a new 3.x release I could make this change against the 3.15 branch too.

@jonaskello jonaskello merged commit 7e03299 into jonaskello:master Feb 23, 2026
4 checks passed
@jonaskello
Copy link
Copy Markdown
Owner

I think we'll do this only for 4.x unless someone explicitly needs it in 3.x.

@XhmikosR XhmikosR deleted the xmr/package-json-files branch February 23, 2026 07:29
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