Skip to content

Conversation

abirambul
Copy link
Contributor

No description provided.

@abirambul abirambul changed the title #20 Добавлен плагин замены автора коммита WIP: #20 Добавлен плагин замены автора коммита Sep 16, 2019
@khorevaa
Copy link
Member

А тесты будут? )) Или у тебя где то тестировалось?

@vkislitsin
Copy link

Приветствую, коллеги.

В коде плагина производится поиск НовогоАвтора среди существующих пользователей Хранилища, причем не всех, а полученных из отчета по версиям, т.е. тех, кто выполнял хотя бы один коммит в Хранилище.
СтрокаПользователя = ТаблицаПользователейХранилища.Найти(НовыйАвтор, "Автор");

В моей практике замена Автора коммита чаще всего требуется для подстановки Пользователя, которого нет в Хранилище.
Например, чтобы авторство для обновлений релизов типовых конфигураций было за пользователем "Компания 1С", а не тем, кто обновлял.

Предложил бы не ограничивать подставляемых Авторов только теми, кто есть в Хранилище и выполнял коммиты в нём.

@abirambul abirambul changed the title WIP: #20 Добавлен плагин замены автора коммита #20 Добавлен плагин замены автора коммита Jan 8, 2021
Исправлено поведение плагина.

Для работы плагина требуется только наличие автора в файле AUTHORS. Наличие пользователя в хранилище не обязательно.
@artbear
Copy link
Member

artbear commented Feb 18, 2021

@khorevaa этот плагин у нас, в команде ИТ-Лаборатории Инфостарта, уже полгода работает активно.

завтра, 19.02, на митапе Инфостарта по CI\CD автор немного расскажет об этом плагине и покажет примеры его работы.

Алексей, чего не хватает, чтобы пулл-реквест был принят?

@khorevaa khorevaa merged commit 7107f74 into oscript-library:develop Feb 18, 2021
@khorevaa
Copy link
Member

Принято! Раз работает то можно и без тестов )

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants