Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update package.json's files list to omit test files and to fix sourcemap-debugging issues #343

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

justingrant
Copy link

@justingrant justingrant commented Aug 30, 2021

Updates package.json to:

  • Include src which enables setting breakpoints in patch-package code when it's not running, and enables you to navigate from breakpoints in VSCode to the corresponding source line that's pointed to by the inline sourcemap. Also prevents an annoyance where every time I debug into patch-package code, VSCode prompts me for the directory location of the corresponding TS source file (which I can't provide because it's not installed).
  • Exclude *.test.js and *.test.ts files that shouldn't be installed via npm install or yarn add because these are test-only files.

I tested this config using npm pack and the results are as expected: test files are missing and src files are present.

Updates package.json to:
* Include `src` which enables setting breakpoints in patch-package code
  when it's not running, and enables you to navigate from breakpoints in
  VSCode to the corresponding source line. Also prevents an annoyance
  where every time I debug into patch-package code, VSCode prompts me
  for the directory location of the corresponding TS source file (which
  I can't provide because it's not installed).
* Exclude *.test.js and *.test.ts files that shouldn't be installed via
  `npm install` or `yarn add`.

I tested this config using `npm pack` and the results are as expected:
test files are missing and src files are present.
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