All streams
Search
Write a publication
Pull to refresh
9
0
Send message

Статья какая-то максимально общая и без подробностей, хотелось бы понять хотя бы примерно что не получился говнокод в одном файле на 5000 строк, и при желании mvp можно было расширить без боли.

В документах 1910 годов еще использовались иероглифы.

Интересная статья, но многовато воды и размышлений. Если сократить раза в 4, было бы самое то.

в уже существующий контейнер

Да нет, контейнер собирается с нуля.
Два RUN'а это два последовательных шага в Dockerfile, env.list и build.sh из чего собирается контейнер, ну а дальше билд, ран и пуш в хранилище. Вроде все по шаблону)
Пока не поняла где путаница произошла)

Привет!

Насколько вам удобен получившийся формат вывода отчёта?

В целом, формат и правда довольно простой, и служит скорее для уведомления о том, что что-то не так. Ваш вариант звучит интересно, можно вполне будет внедрить. Пока все решалось тем, что в проекте можно запустить ktlint, и оттуда уже перейти напрямую к ошибкам.

у вас отдельные docker-контейнеры под сборку APK и под запуск статического анализа

Нет, контейнер один. А из-за чего вы так подумали? Возможно где-то стоит поправить инструкцию для лучшего понимания :)

а почему используете ktlint, а не detekt

Все просто, исторически так сложилось в проекте. В рамках технической задачи мне предложили подключить danger, и выводить туда чего-нибудь полезное. Что-то я даже не подумала про смену ktlint на detekt, и пошла писать велосипед)
С другой стороны, этой статьи могло бы и не быть, если бы я поменяла ktlint на detekt, и просто использовала их плагин. А так в теории, теперь можно выводить вообще что угодно, главное плагин написать.

Спасибо за такой развернутый ответ!

Привет, читала про MVICore, насколько я поняла, вся бизнес-логика инкапсулирована в Feature, и они взаимодействуют друг с другом (а так же их может быть несколько на экран). Хотелось бы увидеть более advanced примеры кода, чем sample у вас в репозитории на гитхабе :) ну или хотя бы на пальцах узнать как конкретно взаимодействуют две фичи на абстрактном примере.

Information

Rating
Does not participate
Registered
Activity