test:verbose script

[brodycj]

to run jest with --verbose flag

which is now used in .travis.yml

I am partly raising this to see if Travis CI works on PRs. I do not see Travis CI checkmarks in https://github.com/wollardj/node-simple-plist/commits/master.

Also it looks like the default branch of this project is develop, which does not have any commits since 2016.

[brodycj]

I would like to add an "off-topic" comment that I am not personally a fan of git commit hooks. Maybe this makes me a bit "old fashioned" but I generally prefer to manually do the "npm test" step before committing. And if we would always use PRs to make functional changes, and check for green build before merging, then we should never have an issue with untested broken code on GitHub.

I use git commit --amend quite a bit to cleanup commit messages after committing code changes, in which case I do not find the git commit hook so helpful.

Of course I could use "--no-verify" to get around the git hook but I would rather avoid the use of this kind of a trick.

[wollardj]

Yeah, I can understand that point of view. Of course you're right that the use of CI workflow makes the git commit hooks more of a belt-and-suspenders approach, but to me it's target is less about keeping PRs stable (that's what TravisCI is for), it's more about keeping individual commits stable.

This project is not a huge one, so it might be less of a concern here. But on larger projects it makes a little more sense to worry about commit-level stability. I'm very in-tune with my human nature to appreciate patterns so I go push the trivial tasks to autopilot and focus on the bigger picture. So with that in mind, I opted to go the same route that I would have taken for a larger project.

Thanks for the thoughts!