Template fixes

[chausovSurfStudio]

В данном ПРе поправлены следующие проблемы:

Changes:

• структура нового проекта из этого шаблона заточена под архитектуру с координаторами
• файлы для CI теперь автоматом добавляются, не надо их потом тянуть извне
• и теперь на PR будут автоматом тесты гоняться, необходимый код для этого добавлен
• обновил fastfile, но тут я не шибко силен, надо обсуждать
• поправил опечатку в swiftlint yaml файле, из-за которой проект из коробки не собирался
• обновил версии для cocoapods и fastlane
• commit-msg хук поправил
• обновил слегка Makefile, там теперь help команда есть ) для таких забывчивых как я
• ну и расширение одно добавил в модуль с тестами, которое нужно для написание тестов на утечку памяти

Notes:

• не уверен, все ли из этого нужно в шаблон включать, может быть что-то планировалось как раз таки добавлять/изменять извне (к примеру, логику в fastfile или доп файлы для поддержания архитектуры с координаторами)
• и не уверен, стоило ли мне писать это сообщение на русском, может на английском тут надо?)
• а почему тут Ваню Сметанина нельзя в ревьюеры добавить?(

[ismetanin]

@chausovSurfStudio

а почему тут Ваню Сметанина нельзя в ревьюеры добавить?(

Потому что я не в тиме уже 😢

Но если тебе важно, то ПР мне нравится, только ты ещё можешь убрать .gitkeep файл из тех директорий, где уже есть какие-то ещё файлы, .gitkeep нужен для того чтобы папка в гит закоммитилась, гит не умеет в пустые папки

И ещё как совет: какие-то спорные моменты можно будет закрыть под опции по типу как в стандартном шаблоне "Use Core Data" выглядит например

[chausovSurfStudio]

@chausovSurfStudio

а почему тут Ваню Сметанина нельзя в ревьюеры добавить?(

Потому что я не в тиме уже 😢

Но если тебе важно, то ПР мне нравится, только ты ещё можешь убрать .gitkeep файл из тех директорий, где уже есть какие-то ещё файлы, .gitkeep нужен для того чтобы папка в гит закоммитилась, гит не умеет в пустые папки

И ещё как совет: какие-то спорные моменты можно будет закрыть под опции по типу как в стандартном шаблоне "Use Core Data" выглядит например

да вот да, надо только разобраться, как это делать)) чтобы под опции закрыть, а то уже от Сани поступило предложение разделить его на два: чистое MVP серфовское и MVP на координаторах

[ismetanin]

Вот тут можно почитать.

Но это муторно всё будет, особенно если будет много опций

[chausovSurfStudio]

в общем, еще немного допилил, из нового:

• теперь два шаблона устанавливаются, один на обычный MVP, второй на MVP с координаторами
• шаблон с обычным MVP юзает шаблоны из студийного репозиторий, в структуре проекта старая добрая папка User Stories и сразу затягивается протокол+реализация для ModuleTransitionable
• шаблон с координаторами имеет выбор storyboard vs xib, для сторибордов - используются шаблоны из студийного репозитория, для ксибов - из его форка в моем репозитории (немного прокачал по код-стайлу + добавил шаблон для координатора, а то его я тоже устал сам руками писать, пусть генерится). Почему? Потому что мне сейчас еще холивара по поводу новых шаблонов не хватало на месяц, кто хочет - пусть юзает студийные со сторибордом, кто не хочет - мои, кто не желает ни те, ни другие - может свои затянуть
• по красоте добавил папку Templates/ в гитигнор
• ну и замечание Вани насчет марка поправил

[chausovSurfStudio]

а пока не смержили: по горячим следам поправил два бага)

• добавил файл Localizable.strings в таргет приложения, а то строки свфитгеном генерились, а значения не цеплялись
• и добавил на Static code analysis стратегию FAIL_WHEN_STAGE_ERROR

[chausovSurfStudio]

@gregoryvit все поправил кроме PR джоба, его бы обсудить
право зарезолвить остальное оставляю за тобой)

[gregoryvit]

@chausovSurfStudio вроде pr джоб обсудили, можно смотреть финально?

[chausovSurfStudio]

@gregoryvit только сейчас руки дошли, поправил PR job, можно смотреть)