Development using git:Quality assurance: Difference between revisions

From Alpine Linux
(Add a git hook to catch common errors early)
 
mNo edit summary
 
(One intermediate revision by one other user not shown)
Line 1: Line 1:
{{Move|Git|All git development articles should be consolidated}}
Before pushing anything to it is good to make sure that:
Before pushing anything to it is good to make sure that:
* The package actually builds
* The package actually builds
Line 26: Line 28:


Install it as .git/hooks/pre-commit and make it executable.
Install it as .git/hooks/pre-commit and make it executable.
= See Also =
*[[Development using git]]
[[Category:Development]]
[[Category:Git]]

Latest revision as of 19:22, 9 December 2018

This page is proposed for moving ...

It should be renamed to Git. All git development articles should be consolidated (Discuss)

Before pushing anything to it is good to make sure that:

  • The package actually builds
  • Commit message is good
  • pkgrel is bumped if needed
  • no whitespace damage (last chars of a line is whitespace)

The following git hook will help you catch some common errors early:

#!/bin/sh

# Redirect output to stderr.
exec 1>&2

git diff --cached --name-only HEAD | grep 'APKBUILD$' | while read f; do
        olddir=$PWD
        cd ${f%/APKBUILD}
        if ! abuild sanitycheck && verify; then
                exit 1
        fi
        cd "$olddir"
done

# If there are whitespace errors, print the offending file names and fail.
exec git diff-index --check --cached HEAD --

Install it as .git/hooks/pre-commit and make it executable.

See Also