Skip to content

Commit

Permalink
Fix allow listed files for what to include in npm package
Browse files Browse the repository at this point in the history
When #296 was merged, the list of files to include in `package.json` was not updated. This PR fixes that.

The `.npmignore` file is also removed because I find it causes some unexpected behaviors like ignoring the contents of `.gitignore`. It's a bit of extra work to maintain the allow listed `files` but it's generally safer.
  • Loading branch information
colincasey committed Feb 23, 2024
1 parent 069dffc commit 26e84f2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
5 changes: 0 additions & 5 deletions .npmignore

This file was deleted.

4 changes: 3 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,9 @@
"types": "./dist/cookie/index.d.ts",
"files": [
"dist/*.js",
"dist/*.d.ts"
"dist/*.d.ts",
"dist/cookie/*.js",
"dist/cookie/*.d.ts"
],
"scripts": {
"build": "npm run clean && tsc",
Expand Down

0 comments on commit 26e84f2

Please sign in to comment.