Как я могу выполнить проверку git pre-commit только на поэтапном контенте?

Предположим, что git status дает следующее:

# On branch X
# Changes to be committed:
#   (use "git reset HEAD <file>..." to unstage)
#
#   modified:   file1.cc
#   modified:   file1.h
#   modified:   file1_test.cc
#   modified:   SConscript
#
# Changes not staged for commit:
#   (use "git add <file>..." to update what will be committed)
#   (use "git checkout -- <file>..." to discard changes in working directory)
#   (commit or discard the untracked or modified content in submodules)
#
#   modified:   file1.cc
#   modified:   tinyxml2 (untracked content)
#

В этом случае только некоторые изменения, внесенные в файл1.cc, были поставлены/индексированы для следующей фиксации.

Я запустил pre-commit script, чтобы выполнить проверку стиля:

#!/bin/bash                                                                                                                                                                     

git stash -q --keep-index

# Do the checks                                                                                                                                                                 
RESULT=0
while read status file
do
    if python ~/python/cpplint.py "$file"; then
        let RESULT=1
    fi
done < <(git diff --cached --name-status --diff-filter=ACM | grep -P  '\.((cc)|(h)|(cpp)|(c))$' )

git stash pop -q

[ $RESULT -ne 0 ] && exit 1
exit 0

Как было предложено здесь, я ставил неустановленные файлы, прежде чем запускать свои проверки стиля, и затем вывожу их. Однако в случае, когда только некоторые изменения в файле были поставлены, это приводит к конфликту слияния, когда я выталкиваю тайник в конце крюка предварительной фиксации.

Что лучше для этого? Я хочу запустить проверку стиля против поэтапной версии файлов, которые должны быть зафиксированы.

Ответ 1

Я бы не стал автоматически использовать git stash в hooks. Я обнаружил, что можно использовать git show ':filename' для получения содержимого скрытого файла.
Вместо этого я использовал следующий подход:

git diff --cached --name-only --diff-filter=ACMR | while read filename; do
    git show ":$filename" | GIT_VERIFY_FILENAME="$filename" verify_copyright \
        || exit $?
done \
    || exit $?

Ответ 2

Замените git stash -q --keep-index на:

git diff --full-index --binary > /tmp/stash.$$
git stash -q --keep-index

... и git stash pop -q с:

git apply --whitespace=nowarn < /tmp/stash.$$` && git stash drop -q
rm /tmp/stash.$$

Это сохранит diff во временном файле и повторит его использование с помощью git apply в конце. Изменения по-прежнему сохраняются в резервном хранилище (который позже удаляется), так что если что-то пойдет не так с повторным применением, вы можете проверить их с помощью git stash show -p без необходимости поиска временного файла.

Ответ 3

Как заблокировать ворота без фиксации

Мы используем этот .git/hooks/pre-commit для проверки пакета синтаксиса атома

Бит ключа

git checkout-index -a --prefix={{temp_dir}}

Он может/не может быть намного медленнее/занимать больше места, чем зашивать, но автоматизация, которая беспорядочна с индексом, кажется неотъемлемо хрупкой. Возможно, должен быть git contrib script, который создает дерево с мягкой/жесткой связью, минимальное пространство только для чтения, временную проверку индекса, облегчая/ускоряя .git/hooks/pre-commit (или, скажем, .git/hooks/pre-commit-index) скриптов, так что полная вторая копия рабочего каталога не требуется, изменяется только рабочий индекс dir- > .

#!/usr/bin/env ruby
require 'tmpdir'
autoload :FileUtils,  'fileutils'
autoload :Open3,      'open3'
autoload :Shellwords, 'shellwords'

# ---- setup

INTERACTIVE         = $stdout.tty? || $stderr.tty?
DOT                 = -'.'
BLOCK_SIZE          = 4096
TEMP_INDEX_DIR      = Dir.mktmpdir
TEMP_INDEX_DIR_REAL = File.realpath(TEMP_INDEX_DIR)

def cleanup
  FileUtils.remove_entry(TEMP_INDEX_DIR) if File.exist? TEMP_INDEX_DIR
end

at_exit { cleanup }

%w[INT TERM PIPE HUP QUIT].each do |sig|
  Signal.trap(sig) { cleanup }
end

# ---- functions

def fix_up_dir_output(data)
  data.gsub! TEMP_INDEX_DIR_REAL, DOT
  data.gsub! TEMP_INDEX_DIR, DOT
  data
end

def sh(*args)
  Open3.popen3(*args) do |_, stdout, stderr, w_thr|
    files = [stdout, stderr]
    until files.empty? do
      if ready = IO.select(files)
        ready[0].each do |f|
          begin
            data = f.read_nonblock BLOCK_SIZE
            data = fix_up_dir_output data
            if f.fileno == stderr.fileno
              $stderr.write data
            else
              $stdout.write data
            end
          rescue EOFError
            files.delete f
          end
        end
      end
    end
    if !(done = w_thr.value).success?
      exit(done.exitstatus)
    end
  end
end

def flags(args)
  skip = false
  r = []
  args.each do |a|
    if a[0] == '-' && !skip
      a.slice! 0
      if a[0] == '-'
        skip ||= a[1].nil?
        a.slice! 0
        r << a unless a.empty?
      else # -[^-]+
        r += a.split ''
      end
    end
  end
  r
end

def less_lint
  args = %w[lessc --lint]
  args << '--no-color' unless INTERACTIVE
  args << 'index.less'
  sh(*args)
end

def ensure_git_commit_signed
  pcmd = `ps -wwp#{Process.ppid} -ocommand=`.chop
  args = flags(Shellwords.split(pcmd)[1..-1])
  return unless (args & %w[gpg-sign S]).empty?
  $stderr.puts 'All git commits must be GPG-signed'
  $stderr.puts "    command: #{pcmd}"
  exit 1
end

# ---- main

# 1. make sure all commits are signed
ensure_git_commit_signed

# 2. check files that are in the index
sh 'git', 'checkout-index', '-a', "--prefix=#{TEMP_INDEX_DIR}/"
Dir.chdir TEMP_INDEX_DIR do
  # 3. make sure all commits contain only legal .less files
  less_lint
end