So far, a pre-commit hook was triggered by default that ran npm test whenever you committed. I personally found this a bit annoying and always removed the hooks manually, but due to the configuration in package.json, they are always reinstalled whenever you invoke npm install.
This PR gets rid of the hooks by removing the pre-commit dependency and the pre-commit field in package.json. This gives us developers a bit more control without sacrificing code quality here on GitHub since the CI action is running anyway, ensuring that the code base passes all tests.
This is just a personal preference, though, if you would like to keep the hook, we could also leave it as is and close this PR :)
So far, a pre-commit hook was triggered by default that ran
npm testwhenever you committed. I personally found this a bit annoying and always removed the hooks manually, but due to the configuration inpackage.json, they are always reinstalled whenever you invokenpm install.This PR gets rid of the hooks by removing the
pre-commitdependency and thepre-commitfield inpackage.json. This gives us developers a bit more control without sacrificing code quality here on GitHub since the CI action is running anyway, ensuring that the code base passes all tests.This is just a personal preference, though, if you would like to keep the hook, we could also leave it as is and close this PR :)