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

первое

А гляньте еще разок на мою вторую попытку

https://github.com/f3ath/release-notes-generator

https://github.com/f3ath/release-notes-generator

По-моему, идейно хорошо.

В php нет традиции интерфейсы как-то декорировать, типа "IRelease", как в С++?

хбз, я подписываю прям InterfaceName

В php нет традиции интерфейсы как-то декорировать, типа "IRelease", как в С++?

Нету соглашения какого-то, насколько я знаю

А я NameInterface

Хм, в джаве у интерфейса никаких префиксов или суффиксов не делают, а у реализации, иногда бывает суффиксы impl или default

Мое мнение что название реализации должно содержать ее суть

а если у тебя один интерфейс и одна имплементация, то возможно смысла в этом интерфейсе нет

а если у тебя один интерфейс и одна имплементация, то возможно смысла в этом интерфейсе нет

Хм, а как делать тогда di?

Хм, а как делать тогда di?

оно и без интерфейсов работает

по крайней мере в спринге

в спринге транзации работают хуже без интерфейсов из-за проксирования обильного

Да, но как только у тебя появится еще одна реализации, в слишком многих местах придётся менять значения.

как только появится еще одна реализация, то тогда нужен интерфейс, да

Да, но как только у тебя появится еще одна реализации, в слишком многих местах придётся менять значения.

ide с этим вполне справляется когда делаешь extract interface

ide с этим вполне справляется когда делаешь extract interface

Не помню такого, и слабо представляю, что она мне конфиги править будет.

https://www.jetbrains.com/help/idea/2016.2/extract-interface.html

Слабо представляю какой профит от того, что в имени интерфейса есть какое-то указание, что это интерфейс

проще навигация в гитхабе)

https://github.com/f3ath/release-notes-generator

https://github.com/f3ath/git-changelog/blob/master/src/Release/BaseRelease.php protected:/

Слабо представляю какой профит от того, что в имени интерфейса есть какое-то указание, что это интерфейс

скорее всего потому, что чаще всего интерфейс == реализация

но всем с универа в голову вбили, все принципы solid

Так в солид ж нет принципа добавляй I в имена интерфейсов

нет, есть принцип программирования от интерфейсов

Да и круто, если в универе принципы солид хотя бы упоминалось))

а когда у тебя есть блабла интерфейс, нужна же блабла реализация

в c# делают IБлаБла, в джаве БлаБлаImpl на реализации

Это всего лишь нейм конвейшин

http://cs.pikabu.ru/images/big_size_comm/2013-05_3/13684203607863.jpg

https://github.com/f3ath/release-notes-generator

https://github.com/f3ath/git-changelog/blob/master/src/Release/SubsequentRelease.php#L12 А почему здесь тег, а не релиз?

https://github.com/f3ath/release-notes-generator

https://github.com/f3ath/git-changelog/blob/master/src/Git/Git.php#L6 я бы назвал эту штуку типа ShellGit, потому что как вариант может появиться GitGateway через какие-нибудь биндинги к сишке

И какой смысл давать возможность не передавать Shell и RepoDetector

https://github.com/f3ath/git-changelog/blob/master/src/Release/BaseRelease.php protected:/

Пояснишь?

Пояснишь?

protected это плохо

в 99% случаев

даже Страуступ говорит что "зря пошли на поводу и впилили protected в c++"

https://github.com/f3ath/git-changelog/blob/master/src/Release/SubsequentRelease.php#L12 А почему здесь тег, а не релиз?

Потому что в терминах гита и гитхаба это тег

Потому что в терминах гита и гитхаба это тег

а в терминах пользователя?

protected это плохо

Как надо?

Как надо?

private

)

private по дефолту, public если у тебя тупо контейнер с данными, и никогда не делать protected

только когда рефакторишь легаси можно

но если пишешь с нуля лучше просто не использовать protected

private

Не понимаю. :( Можешь оформить это как пулреквест?

Не понимаю. :( Можешь оформить это как пулреквест?

просто избавься от наследования и все будет хорошо

Потому что в терминах гита и гитхаба это тег

релиз ссылается на тег и предыдущий релиз

блин, почему люди так любят совать гейтвеи прямо в сущности?

Я наслушался теории настолько, что она из ушей лезет уже. Как надо?

содержит тег*

Я наслушался теории настолько, что она из ушей лезет уже. Как надо?

запрети себе использовать ключевое слово extends

вообще

Чуваки, я вас всех безмерно уважаю и люблю. Покажите в реальности, как надо сделать? Пулреквестом. Пожалуйста.

Чуваки, я вас всех безмерно уважаю и люблю. Покажите в реальности, как надо сделать? Пулреквестом. Пожалуйста.

форкнул. Жди

так

глянул код внимательнее

начнем со злопалучного protected и зачем ты его юзаешь

сначала уберем наследование, которое нам не нужно

@f3ath как проверить работу приложения?

тесты ты так и не написал

даже интеграционные

даже Страуступ говорит что "зря пошли на поводу и впилили protected в c++"

в jdk много где юзается, например в io стримах

в jdk много где юзается, например в io стримах

это не говорит о том что это хорошо.

хотя в io стримах возможно и нужно

это не говорит о том что это хорошо.

просто по-другому сложно было бы сделать

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

@f3ath > $conf = Config::fromJsonFile();

ужас

вот это плохая статика

грязная

с сайд эффектами

ее невозможно нормально протестировать

https://twitter.com/KentBeck/status/812703192437981184

хорошая метафора)

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

не понятно почему git gateway и tag передаются явно как зависимости а принтер через дабл диспатч

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

ух