git pre-commit hook同时格式化和重新添加文件

时间:2015-06-25 18:02:29

标签: git formatting githooks

我们目前正在使用git hook(下面)在允许用户提交之前在源代码上运行astyle。这有一个警告,用户必须提交,将其代码格式化,然后再次提交,这有点令人讨厌。理想情况下,我们希望钩子格式化代码,然后在原始提交中包含格式化代码。我已经尝试重新添加更改的文件,但它会导致ref错误(显然)。我也试过在预提交钩子中获取历史记录并尝试退出钩子并重新运行git commit命令而没有运气。

# Run astyle on changed .cs files, ignoring $ignored
res=$(exec git diff --cached --name-only | \
    grep -Ev $ignored | \
    xargs astyle --options=conf/astylerc | \
    tail -n 1)
num_formatted=$(echo $res | cut -b 1) # We are only interested in the number preceeding 'formatted'.
[[ $num_formatted -ne 0 ]] && echo "WARNING: Code has been automatically formatted. Please re-add and re-commit" && exit 1 || echo "No code to format! Continuing commit"

有没有人有任何想法?

3 个答案:

答案 0 :(得分:4)

在你的预提交钩子中,你需要添加你的文件,所以如果你的钩子是这样的:

#!/bin/bash
echo 1 > file
exit 0

然后您需要修改它以添加:

#!/bin/bash
echo 1 > file
git add file
exit 0

要获取所有已修改文件的列表,可以使用git-ls-files:

git ls-files -m

但是,如果您只是从代码中获取修改了哪些文件的列表,或者只是再次添加所有文件,那就更好了。 git diff-tree -r --name-only --no-commit-id <tree-ish>应该可以帮助您获取所有文件的列表。

基本上,在修改之后再次添加文件是有效的,因为直到你的预提交钩子运行之后才会发生提交,所以在那个工作树中提交的任何内容都会被提交。

答案 1 :(得分:2)

this回答后

已修改

您可以在钩子中格式化并添加文件。问题是您可能对分阶段文件进行了非分段修改。要以一种干净的方式执行此操作,您可以将索引中的文件作为tmp,格式化tmp并使用格式化的tmp替换索引中的条目。这是解决问题的方法:

# Regexp for grep to only choose some file extensions for formatting
exts="\.\(ext\|ext2\)$"

# The formatter to use
formatter=`which your_formatter`

# Check availability of the formatter
if [ -z "$formatter" ]
then
  1>&2 echo "$formatter not found. Pre-commit formatting will not be done."
  exit 0
fi

# Format staged files
for file in `git diff --cached --name-only --diff-filter=ACMR | grep $exts`
do
  echo "Formatting $file"
  # Get the file from index
  git show ":$file" > "$file.tmp"
  # Format it
  "$formatter" -i "$file.tmp"
  # Create a blob object from the formatted file
  hash=`git hash-object -w "$file.tmp"`
  # Add it back to index
  git update-index --add --cacheinfo 100644 "$hash" "$file"
  # Remove the tmp file
  rm "$file.tmp"
done

# If no files left in index after formatting - fail
ret=0
if [ ! "`git diff --cached --name-only`" ]; then
  1>&2 echo "No files left after formatting"
  exit 1
fi

答案 2 :(得分:1)

您可以使用 here 描述的技术仅存储未暂存的更改。然后仅对分阶段的更改运行格式化程序并弹出存储。在 pre-commit 钩子下方使用 clang-format-diff

#!/bin/sh

# stash unstaged changes
git commit --no-verify -m 'Save index'
old_stash=$(git rev-parse -q --verify refs/stash)
git stash push -m 'Unstaged changes'
new_stash=$(git rev-parse -q --verify refs/stash)
git reset --soft HEAD^

# format staged changes
git diff -U0 --no-color --staged HEAD -- '*.java' | $PWD/clang-format-diff.py -i -p1

git add -u
if [ "$old_stash" != "$new_stash" ]; then # if unstaged changes were stashed reapply to working tree
    git stash pop
fi
exit 0