Skip to content

#20 Добавлен плагин замены автора коммита #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Feb 18, 2021

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