Cover Your Ass Engineering - о да :-)

попытка #3 написать что-то объектное https://github.com/f3ath/git-changelog

https://github.com/f3ath/git-changelog/blob/master/src/Git/Commit.php#L23

мне кажется этому место в том месте, где ты лог читаешь

https://github.com/f3ath/git-changelog/blob/master/src/Changelog.php#L20

тут можно было бы завернуть это добро в 2 стратегии

что бы избавиться от if-а

уберем лишний тест кейс

тут можно было бы завернуть это добро в 2 стратегии

мне он тоже не нравится, но я нешмогла

> но я нешмогла ну... просто два класса имплементящих что-то вроде ChangesetProvider

и третий класс который будет выбирать нужный по параметру

но в целом могу сказать что выглядит лучше чем в прошлый раз

еще не заглядывал в тесты

тестов мало

Git не покрыт

но в целом могу сказать что выглядит лучше чем в прошлый раз

спасибо. значит не зря

ну во всяком случае рефакторить меньше)

https://github.com/f3ath/git-changelog/blob/master/src/Versions.php#L22

вот эта штука слегка смущает

чуть-чуть совсем

ну не фабрику же городить ради этого

а тебе фабрика и не нужна, тебе нужен какой-то процессор версий

ну или фабрика некой VersionCollection

если упарываться

что бы система типов описывала взаимодействие логики

тебя смущает plural название?

не, меня смущает что в конструктор может придти что угодно

может

но судя по статическому методу должен придти массив чего-то в строго определенном формате

то есть мы допускаем нарушение инварианта

но semver будет орать, если там лажа

но semver будет орать, если там лажа

да, возможно)

ну мол в таком ключе скорее всего все ок

а из хороших решений что-то есть?

в смысле у меня в коде

RevisionRangeInterface норм

(еще бы без суффиксов но я не буду спорить по coding style)

ну мол ты хорошо изолировал эту хрень

https://github.com/f3ath/git-changelog/blob/master/src/Changelog.php#L31

вот эту штуку бы вынести в какой-нибудь форматтер

но это мелочи

это да

блин напомни завтра

перевод?

перевод?

посижу поперевожу сейчас и пойду спать

просто хотелось с тобой как раз на примере твоего кода про SOLID чутка поболтать)

добро

у тебя там есть маленькие и простые примеры когда ты например SRP нарушил и парочка где ты их соблюдаешь

на примерах все же проще

https://github.com/f3ath/git-changelog/blob/master/src/GitGateway.php

вот этот класс мне не нравится

эдакий фасад-адаптер

без четкой ответственности

но я хз как без него

проанализируй где и зачем ты его юзаешь

чтобы разгрузить Changelog

ну то есть "там-то что бы версию забрать"

там-то что бы чейнджлог забрать между двумя ревизиями

просто потому что без него сложно тестировать

просто потому что без него сложно тестировать

это мелочи, я предлагаю тебе добавить чуть-чуть interface segregation

https://github.com/f3ath/git-changelog/blob/master/src/GitGateway.php#L61

вот эта штука не понятна

в целом он переводит с терминов гита в термины чейнжлога

коммит -> изменение

ERROR: S client not available

в целом он переводит с терминов гита в термины чейнжлога

не не... посмотри зачем он используется там где используется

тег -> версия

ну у него много задач

ладно) завтра)

оке )

> ну у него много задач исходи из позиции "по каким причинам реализация может поменяться" а не по задачам

я из этого и исходил

получается, что "ни по каким"

я не вижу причин для его изменений

поменялся способ работы с git (с вызова git через exec на работу с какой-нибудь библиотечкой сишной интегрированной)

это задача Git

тут все ок

это задача Git

да я ж не спорю)

getDiffUrl

вот метод например

этот тоже изолирован

этот тоже изолирован

что значит изолирован?

ну у него нет причин меняться

о вот к слову

смотри

public function getDiffUrl(string $rev1, string $rev2): string { $remote_url = $this->git->getRemoteUrl($this->remote); $remote = $this->remote_factory->createByUrl($remote_url); return $remote->getDiffUrl($rev1, $rev2); }

поменяется фабрика

у тебя во всем классе только этот метод использует remote_factory

добавится новый Remote\BitBucket

да

а хотя не, он юзает ->git

https://github.com/f3ath/git-changelog/blob/master/src/GitGateway.php#L39

а это чего?

получить дату коммита

тега

ну или версии в нашем случае