такое впечатление, что это отписка преподу на экзамене.
хоть сами бы проверили код перед тем как пушить в общую либу 😐
и имена одного и того же разные в одинаковых фрагментах
и несоблюдение общего стиля
и вообще критикал баги в поведении навигатора!!!
вы разбирались как Чичероне работает? или просто хотели приобщиться к проекту?
пожалуйста, проявите уважение к проекту! каждый мр проверьте и вымучайте на все кейсы!
вы видели спеку команды бекТу?
пока МР - ломает все, глупо показывая, что "вот, удалось вызвать команду на aac навигаторе". код на коленке! такие нельзя предлагать как МР!
кроме того, есть ОЧЕНЬ важный пункт при использовании такого навигатора: надо четко описать, что используя команды Чичероне нельзя описывать поведение переходов на xml графе!
всякие app:popUpToInclusive="false"
это легко все сломает!
"имена одного и того же разные в одинаковых фрагментах" - если вы про btn_* , то да есть такое. Но ведь нигде не написаны требования к кода стайлу, тогда надо какой-то contrubution guide заводить.
"вообще критикал баги в поведении навигатора!!!" - Если вы про асинхронные транзакции фрагмент менеджера, то исправим, ок.
"вы разбирались как Чичероне работает? или просто хотели приобщиться к проекту?" - Если бы не разобрался, не писал бы навигатор )) Про приобщиться к проекту не понял вопроса. Можно также сделать отдельный репозиторий с расширением для cicerone и паблишить на мавен как отдельный артефакт, но логичнее конечно в основном репозитории как модуль.
"пожалуйста, проявите уважение к проекту! " - пожалуйста, проявляйте уважение к контрибьютерам и пишите ТОЛЬКО конструктивные замечания, без перехода на личности.
"кроме того, есть ОЧЕНЬ важный пункт при использовании такого навигатора: надо четко описать, что используя команды Чичероне нельзя описывать поведение переходов на xml графе!" - а смысл тогда от навигатора? Только в том, чтобы визуально граф перехода видеть?
"всякие app:popUpToInclusive="false"
это легко все сломает!" - через команду forward, что в итоге вызывает navigate у navcontroller работает. Понятно, что если пользователю либы позволить всякие таки флаги, то он может что-нибудь нехорошее натворить. Ну тут я согласен, надо подумать что какие флаги можно использовать, а какие не стоит.
Про доку согласен, надо сделать ее в markdown как на главной странице.