Вот небольшой сценарий крючков, который делает то, что вы хотите: Необходимо настроить 2 вещи:

• незаконные _ суффиксы : список python со всеми суффиксами, которые должны прервать фиксацию
• cmdSVNLOOK : путь к программе svnlook

import sys
import subprocess 
import re

#this is a list of illegal suffixes:
illegal_suffixes = ['.exe','.dll']

# Path to svnlook command:
cmdSVNLOOK="/usr/bin/svnlook";

def isIllegalSuffix(progname):
    for suffix in illegal_suffixes:
        if (ptFilename.endswith(suffix)):
            return True
    return False

######### main program ################
repopath = sys.argv[1]
transact = sys.argv[2]

retVal = 0
svninfo = subprocess.Popen([cmdSVNLOOK, 'changed', '-t', transact, repopath], 
                                                        stdout = subprocess.PIPE, stderr=subprocess.PIPE)
(stdout, stderr) = svninfo.communicate();

prog = re.compile('(^[ACUDRM_])[ACUDRM]*\s+(.+)')  # regex for svnlook output
for line in stdout.split("\n"):
    if (line.strip()!=""):
        match=re.search(prog, line.strip())
        if match:
            mode = match.group(1) 
            ptFilename = match.group(2)
            if mode == 'A' and isIllegalSuffix(ptFilename): 
              retVal = 1
              sys.stderr.write("Please do not add the following ")
              sys.stderr.write("filetypes to repository:\n")
              sys.stderr.write(str(illegal_suffixes)+"\n")
              break
        else:
            sys.stderr.write("svnlook output parsing failed!\n")
            retVal = 1
            break
    else:
        # an empty line is fine!
        retVal = 0
sys.exit(retVal)

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

Слышали ли вы о acts_as_state_machine или каких-либо других подобных решениях?

http://github.com/rubyist/aasm

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

Это позволяет вам иметь столько логики, сколько вам нужно о том, когда вещи отправляются, если вам это нужно. Может быть чрезмерным, но может быть очень полезным. Я предлагаю, потому что вы хотите отправить электронное письмо, когда задача «завершена», которое звучит так, как будто это может быть тип состояния или столбец состояния в вашей модели задачи.

Вы правы, первый способ не является хорошим подходом. Наблюдатели - мой предпочтительный путь, по нескольким причинам.

Во-первых, если вы используете TDD (тестовую разработку), вы можете отключить наблюдателей, чтобы более чисто протестировать модель без каждого создания, запускающего создание почтовика. Затем можно отдельно выполнить единичное тестирование почтовика и наблюдателя.

Во-вторых, идея отделить отзывы создает более чистый кодекс. Обратные вызовы на самом деле не являются частью вашей модели, это события. Модель содержит функции и атрибуты, необходимые для выполнения самой модели, а обратные вызовы (реализованные с помощью наблюдателей) являются отдельными обработчиками событий.

Я не думаю, что ваш второй вариант «плохой» или менее профессиональный. В любом случае он работает до тех пор, пока он находится на уровне модели, вместо контроллеров или (что еще хуже) видов.