Skip to content

Conversation

@nixel2007
Copy link
Member

(Смена ветки-источника)

Реализовано через необязательный ПараметрФлаг --remove-orig-bin-files команды --git-precommit.
Протестировано при ручном запуске из командной строки и при запуске через precommit самого git.
Если необходимы какие-то юнит-тесты на данный функционал, прошу помочь с написанием/идеями.
Требуется библиотека cmdline от 21.10.2015 и новее.

Fix #41

…файлов внешних отчетов и обработок из индекса git
@artbear
Copy link
Member

artbear commented Nov 16, 2015

По коду вопросов нет.

По тестам предлагаю скопировать тест Тест_ДолженОбработатьИзмененияИзГитДляКаталогаСВложеннымиКаталогами и немного доработать его:
1 подменить файл pre-commit, добавив флаг --remove-orig-bin-files
2 в новом тесте проверить удаление исходного файла

@artbear artbear self-assigned this Nov 16, 2015
@artbear artbear added this to the 1.0.0-oscript-port milestone Nov 16, 2015
@nixel2007
Copy link
Member Author

@artbear Каким образом лучше сделать подмену файла? После копирования прямо дописать в файл или положить второй файл pre-commit с флагом?

@artbear
Copy link
Member

artbear commented Nov 16, 2015

После копирования дописать в файл

@nixel2007
Copy link
Member Author

Добавил тест. За основу брал версию теста Тест_ДолженОбработатьИзмененияИзГитДляКаталогаСВложеннымиКаталогами с правками из #58

artbear added a commit that referenced this pull request Nov 27, 2015
Fix #41 В команду --git-precommit добавлена функциональность по удалению бинарных файлов обработок при коммите
@artbear artbear merged commit 710bf28 into xDrivenDevelopment:develop Nov 27, 2015
@nixel2007 nixel2007 deleted the issue-41 branch November 29, 2015 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants