Add .npmignore to exclude __tests__ from publish, remove unused deps #24
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR makes two changes to clean up the publish configuration:
.npmignorefiles to all 20 packages - Each file contains__tests__to exclude test directories from npm publishworkspacesfield from package.json (redundant with pnpm-workspace.yaml)cpy-cli,graphile-test,graphql-tag(only referenced in READMEs, no actual imports)pnpm-lock.yamlto sync with the dependency removalsReview & Testing Checklist for Human
pnpm pack --dry-runin a package directory to confirm__tests__is excluded from the tarballworkspacesfield in package.json (lerna.json has"npmClient": "pnpm"so it should use pnpm-workspace.yaml)Recommended test plan: Pick one package (e.g.,
packages/verify), runpnpm pack --dry-runand verify the output excludes__tests__/but includes the expected files (deploy/, verify/, revert/, sql/, LICENSE, Makefile, README.md, pgpm.plan, *.control).Notes
The
.npmignoreapproach excludes__tests__but doesn't explicitly whitelist only the desired files. Other files likejest.config.jswill still be included in publish. If stricter filtering is needed, consider using the"files"field in each package's package.json instead.Link to Devin run: https://app.devin.ai/sessions/8d0bb3e7eee3456788d7919f36fd1880
Requested by: Dan Lynch (@pyramation)