Skip to content
New issue

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

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

Already on GitHub? # to your account

fix: should not publish tap-snapshot folder #471

Closed
wants to merge 1 commit into from

Conversation

ruyadorno
Copy link
Contributor

- Add `tap-snapshot` folder to `.npmignore`
- Fixes npm#461
@ruyadorno ruyadorno requested a review from a team as a code owner November 11, 2019 18:05
@XhmikosR
Copy link
Contributor

This does not fix all redundant files, though.

@claudiahdz claudiahdz added semver:patch semver patch level for changes Release 6.x work is associated with a specific npm 6 release labels Nov 12, 2019
claudiahdz pushed a commit that referenced this pull request Nov 13, 2019
- Add `tap-snapshot` folder to `.npmignore`
- Fixes #461

PR-URL: #471
Credit: @ruyadorno
Close: #471
Reviewed-by: @claudiahdz
@claudiahdz claudiahdz mentioned this pull request Nov 18, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Release 6.x work is associated with a specific npm 6 release semver:patch semver patch level for changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] tap-snapshots in the npm package
3 participants