moxy-community / Moxy

Moxy is MVP library for Android with incremental annotation processor and ktx features
MIT License
324 stars 33 forks source link

Add tests for DailyPicturePresenter and solve some DI problems #115

Closed hram closed 4 years ago

aasitnikov commented 4 years ago

Получилось на сей момент 5 коммитов с сообщением "fixes after code review", может их все сквошнуть в один самый первый коммит?

hram commented 4 years ago

Получилось на сей момент 5 коммитов с сообщением "fixes after code review", может их все сквошнуть в один самый первый коммит?

пожалейте меня пожалуйста я почитал как это делается и ужаснулся никогда не думал что кто то этим пользуется на то они и коммиты чтоб по ним была история

aasitnikov commented 4 years ago

на то они и коммиты чтоб по ним была история

Вообще да, но с исторической точи зрения какую информацию несет, например, третий коммит "fixes after code review"? Никакой полезной. Можно было бы оставить один коммит со всеми фиксами, это норм. Способов есть много - git rebase -i самый удобный, git reset --soft HEAD~5 && git commit самый быстрый Но если не хочется по каким-то причинам, то можно оставить

aasitnikov commented 4 years ago

@hram Евгений, там остался один последний вопрос по интерактору, давайте определимся с ним и вмёржим PR

hram commented 4 years ago

@hram Евгений, там остался один последний вопрос по интерактору, давайте определимся с ним и вмёржим PR

Добрый день. Простите только сегодня заметил сообщение. Я не совсем понял, что за вопрос остался по интерактору. Если по поводу интерфейса так я постарался подробно и аргументировано отстоять свою позицию. Дальше аргументировать у меня нет желания.

Я предложил своё видение решения проблемы. Моё мнение не изменилось. Если не устраивает ПР то в этом нет ничего плохого. Я удалю его или если у вас есть такая возможность можете сами удалить.