rollup-plugin-sass icon indicating copy to clipboard operation
rollup-plugin-sass copied to clipboard

Published 20 hours ago • verified publisher icon elycruz

chore: move npm test to pre-push hook

Open marcalexiei opened this issue 1 year ago • 0 comments

While rebasing #143 I noticed that now tests are run each time a commit is performed.

Someone (me 😅) could using commits as a backup of a work in progress. E.g.:

  • make an incremental commit using --amend
  • making a certain amount of commits and squash them together before push

In the current situation I would find myself unable to commit because some tests might be failing. To be able to commit I have to add --no-verify flag to the commit command, which IMHO is not ideal.

This PR basically move the npm test from pre-commit to pre-push hook. I think is better to check that tests are working before push instead of every commit.

marcalexiei avatar Jul 06 '24 06:07 marcalexiei