fix: do not include .nyc_output in published files by andrewhampton ยท Pull Request #851 ยท commitizen/cz-cli
Navigation Menu
{{ message }}
commitizen / cz-cli Public
- Notifications You must be signed in to change notification settings
- Fork 567
Merged
LinusU merged 1 commit intocommitizen:masterfrom
Nov 8, 2021Merged
fix: do not include .nyc_output in published files#851
LinusU merged 1 commit intocommitizen:masterfrom
fix: do not include .nyc_output in published files#851
LinusU merged 1 commit intocommitizen:masterfrom
Conversation
Copy link
Contributor
andrewhampton
commented
Sep 30, 2021
andrewhampton
commented
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
andrewhampton
mentioned this pull request
LinusU
merged commit
68c377b
into
commitizen:master
Copy link
github-actions
bot
commented
Jul 17, 2022
github-actions bot commented
Jul 17, 2022๐ This PR is included in version 4.2.5 ๐
The release is available on:
Your semantic-release bot ๐ฆ๐
github-actions
bot
added
the
released
label
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment