Хук SVN Pre-commit для временно закомментированного кода (в Java)?

Question

Просто со мной случилось, что я закомментировал строку кода для целей тестирования и закончил тем, что проверил код обратно в хранилище с закомментированной строкой.

Например, я установил правило для работы в EclipseRCP на одной линии.При разработке новой функции я закомментировал эту строку, чтобы правило не устанавливалось.

Через пару часов новая функция была завершена и готова к передаче в SVN.Хотя я много чего меняю, я обычно просто быстро перепроверяю свои изменения, выполняя синхронизацию с репозиторием (в Eclipse) и просматривая все классы, выполняя какую-то проверку здравомыслия (т.е. спрашивая себя, действительно ли я касалсяэтот класс?).Но я не проверяю каждую строку.

Так что в этом случае я не запомнил закомментированную строку, и она была передана в SVN.

Теперь мне было интересно, смогу ли я найти какой-нибудь автоматизм для предотвращения подобных ошибок.Я думал о том, чтобы сделать шаблон кода или тег задачи и пометить закомментированную строку этим.А затем проверьте наличие этого тега задачи в хуке предварительной фиксации SVN.

Я уже некоторое время гуглю, но ничего не смог найти.Кто-нибудь знает способ сделать это?Или, может быть, что-то, что близко к этому и просто нуждается в небольших изменениях, чтобы достичь того, что я ищу?

Answer 1

Заимствование из идеи Спонга ... как насчет того, чтобы добавить какой-либо индикатор в комментарий (FIXME), если его нужно раскомментировать. Хук Pre-commit может выполнить svnlook diff и искать FIXME и предотвратить коммит.

Answer 2

Вы можете проверить такие вещи в хуке перед фиксацией, но я бы предложил сначала использовать ветку и проверить в конце, прежде чем слиться в транк или другую линию интеграции. С другой стороны, я бы предложил использовать модульные тесты для их запуска, и некоторые из них должны быть красного цвета, основываясь на закомментированном коде. И, наконец, что не менее важно помнить о пропущенном удаленном комментарии и делать другой коммит? И что мне приходит в голову: почему бы не использовать @TODO и т. Д. Пометить в Eclipse или лучше @COMMENT и взглянуть на список задач в Eclipse?

Answer 3

Открыть представление задач в Eclipse.Тогда он покажет вам, где у вас есть FIXME в комментариях:

// FIXME - uncomment this before checking in
// importantThing.execute();