Third party report code conventions #373
Conversation
maxsurovtsev
requested review from
matzuk,
AzamatCherchesov,
eakurnikov and
RuslanMingaliev
as code owners
|
|
maxsurovtsev
force-pushed
the
third-party-report
branch
from
ad11346
to
4decf60
Compare
| @@ -22,9 +23,11 @@ dependencyResolutionManagement { | |||
| } | |||
| } | |||
|
|
|||
| val kasperskyRepoUrl: String? by settings | |||
There was a problem hiding this comment.
Это какой-то секрет? Откуда он будет тянуться при публикации?
There was a problem hiding this comment.
Да, безопасность запрещает публиковать ссылку на внутренний репозиторий
There was a problem hiding this comment.
@RuslanMingaliev для релиза мы будем скачивать с локальных ресурсов патч, накатывать его и отправлять отчеты для получения апрува на релиз
| @@ -0,0 +1,3 @@ | |||
| plugins { | |||
There was a problem hiding this comment.
Почему это convention плагин? Не вижу никакой настройки
There was a problem hiding this comment.
Вот тут как раз прописываются зависимости, нужные для конвеншна
Этот файл вместе с convention.air.gradle.kts и convention.third-party-report.gradle.kts оставлены пустыми, чтобы проект продолжал собираться.
| @@ -0,0 +1 @@ | |||
| // Please do not remove this file. This file is used for new releases. | |||
There was a problem hiding this comment.
Не понимаю, для чего вообще этот PR :)
Добавь, пожалуйста, контекста к описанию PR в формате "Какую проблему решал - Что сделал - Что дальше"
There was a problem hiding this comment.
Обновил описание
|
Guys I think we should use English as it is a an international open-source :) We have non-russian speaking contributors. |
|
@RuslanMingaliev hey! |
# Conflicts: # build.gradle.kts # gradle/libs.versions.toml
В этом ПРе создается модуль с конвеншнами для сборки third-party-code отчетов.
Сами конвеншны пустые и ссылка на репозиторий хранится в пропертях из-за политики безопасности в Лаборатории Касперского.
Генерируем мы эти отчеты в контуре ЛК следующим образом: