You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
|
|
|
---
|
|
|
|
title: "Git tips - lint + test pre-commit hook"
|
|
|
|
date: 2020-01-01T21:00:39Z
|
|
|
|
tags: [git, bash]
|
|
|
|
---
|
|
|
|
|
|
|
|
A nice last minute sanity check I use is a `pre-commit` hook that auto runs test and lint commands from a `makefile` or
|
|
|
|
`package.json` if they're found:
|
|
|
|
|
|
|
|
```bash
|
|
|
|
#!/usr/bin/env bash
|
|
|
|
|
|
|
|
if [ -f "$PWD/makefile" ] && [ ! -z "$(cat $PWD/makefile | grep '^lint:')" ]; then
|
|
|
|
echo "running make lint"
|
|
|
|
make lint
|
|
|
|
elif [ -f "$PWD/package.json" ] && [ ! -z "$(cat $PWD/package.json | grep "^\"lint\":")" ]; then
|
|
|
|
echo "running npm run lint"
|
|
|
|
npm run lint
|
|
|
|
fi
|
|
|
|
|
|
|
|
if [ -f "$PWD/makefile" ] && [ ! -z "$(cat $PWD/makefile | grep '^test:')" ]; then
|
|
|
|
echo "running make test"
|
|
|
|
make test
|
|
|
|
elif [ -f "$PWD/package.json" ] && [ ! -z "$(cat $PWD/package.json | grep "^\"test\":")" ]; then
|
|
|
|
echo "running npm run test"
|
|
|
|
npm run test
|
|
|
|
fi
|
|
|
|
```
|
|
|
|
|
|
|
|
If the test or lint command fails then the `git commit` command fails. If I absolutely need to commit something in spite
|
|
|
|
of the lint/test results I can do `git commit --no-verify` to skip the `pre-commit` hook.
|
|
|
|
|