Git Pre-Commit-Hook: Geänderte / hinzugefügte Dateien

Ich schreibe einen Pre-Commit-Hook. Ich möchte rennenphp -l gegen alle Dateien mit der Erweiterung .php laufen. Ich stecke jedoch fest.

Ich muss eine Liste der neuen / geänderten Dateien erhalten, die bereitgestellt werden. gelöschte Dateien sollten ausgeschlossen werden.

Ich habe versucht, git diffund zu verwenden git ls-files, aber ich denke, ich brauche hier eine Hand.

git diff --cached --name-status zeigt eine Zusammenfassung der Inszenierungen an, sodass Sie entfernte Dateien einfach ausschließen können, z.

M       wt-status.c
D       wt-status.h

Dies zeigt an, dass wt-status.c geändert und wt-status.h im Staging-Bereich (Index) entfernt wurde. So überprüfen Sie nur Dateien, die nicht entfernt wurden:

steve@arise:~/src/git <master>$ git diff --cached --name-status | awk '$1 != "D" { print $2 }'
wt-status.c
wt-status.h

Sie müssen durch zusätzliche Rahmen springen, um mit Dateinamen mit Leerzeichen umzugehen (-z Option zum Git-Diff und einige interessantere Analyse).

Ein etwas übersichtlicherer Weg, um dieselbe Liste zu erhalten, ist:

git diff --cached --name-only --diff-filter=ACM

Dies gibt die Liste der Dateien zurück, die überprüft werden müssen.

Es ist jedoch php -lmöglicherweise nicht richtig, nur Ihre Arbeitskopie zu verwenden. Wenn Sie ein partielles Commit durchführen, dh nur eine Teilmenge der Unterschiede zwischen Ihrem aktuellen Arbeitssatz und dem HEAD für das Commit auswählen, wird der Test für Ihren Arbeitssatz ausgeführt, es wird jedoch ein Commit zertifiziert, das auf Ihrem Commit noch nie vorhanden war Scheibe.

Um es richtig zu machen, sollten Sie das gesamte inszenierte Bild in einen temporären Bereich extrahieren und dort den Test durchführen.

rm -rf $TEMPDIR
mkdir -p $TEMPDIR
git checkout-index --prefix=$TEMPDIR/ -af
git diff --cached --name-only --diff-filter=ACM | xargs -n 1 -I '{}' \bin\echo TEMPDIR/'{}' | grep \\.php | xargs -n 1 php -l

Siehe Erstellen eines besseren Pre-Commit-Hooks für Git weitere Implementierung finden .

Keine der Antworten hier unterstützt Dateinamen mit Leerzeichen. Der beste Weg dafür ist, das -zFlag in Kombination mit hinzuzufügenxargs -0

git diff --cached --name-only --diff-filter=ACM -z | xargs -0 ...

Dies wird von git in integrierten Beispielen angegeben (siehe .git / hooks / pre-commit.sample ).

Folgendes verwende ich für meine Perl-Checks:

#!/bin/bash

while read st file; do
    # skip deleted files
    if [ "$st" == 'D' ]; then continue; fi

    # do a check only on the perl files
    if [[ "$file" =~ "(.pm|.pl)$" ]] && ! perl -c "$file"; then
        echo "Perl syntax check failed for file: $file"
        exit 1
    fi
done < <(git diff --cached --name-status)

für PHP wird es so aussehen:

#!/bin/bash

while read st file; do
    # skip deleted files
    if [ "$st" == 'D' ]; then continue; fi
    # do a check only on the php files
    if [[ "$file" =~ ".php$" ]] && ! php -l "$file"; then
        echo "PHP syntax check failed for file: $file"
        exit 1
    fi
done < <(git diff --cached --name-status)

git diff --cached ist nicht ausreichend, wenn der Festschreibungsaufruf mit dem Flag -a angegeben wurde, und es gibt keine Möglichkeit festzustellen, ob dieses Flag in den Hook geworfen wurde. Es wäre hilfreich, wenn die zu feststellenden Argumente dem Haken zur Prüfung zur Verfügung stehen sollten.