Skip to content

chore: remove .npmignore to publish test/ directory - #263

Open
Tony133 wants to merge 1 commit into
mainfrom
chore/remove-npmignore
Open

Tony133 wants to merge 1 commit into
mainfrom
chore/remove-npmignore

Conversation

@Tony133

@Tony133 Tony133 commented Oct 19, 2026

Copy link
Copy Markdown
Member

Proposal:

Remove .npmignore so the test/ directory is included in the published npm package.

Why
.npmignore listed test/, so the tests were not shipped with the package. Deleting the file makes npm fall back to .gitignore, which does not exclude test/.

Verification

  • npm pack --dry-run now lists the test/ directory in the tarball
  • No unwanted files (IDE config, coverage, etc.) are included

Notes:

@Tony133
Tony133 requested a review from a team October 19, 2026 16:47

This branch has not been deployed

No deployments
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.

1 participant