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: do not include .nyc_output in published files #851

Merged
merged 1 commit into from
Nov 8, 2021

Conversation

andrewhampton
Copy link
Contributor

I noticed in a PR review for a bump from 4.2.3 to 4.2.4 that there were
810 changed files. One was packages.json, the other 809 were
.nyc_output json files. This makes it particularly tough to do a
thorough security review.

fixes #730

I noticed in a PR review for a bump from 4.2.3 to 4.2.4 that there were
[810 changed files][1]. One was packages.json, the other 809 were
.nyc_output json files. This makes it particularly tough to do a
thorough security review.

[1]: https://app.renovatebot.com/package-diff?name=commitizen&from=4.2.3&to=4.2.4#d2h-425221

fixes commitizen#730
@github-actions
Copy link

🎉 This PR is included in version 4.2.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

A lot of unnecessary files are included in the npm package
2 participants