AA
а то ведь как же без паттернов
Size: a a a
2020 August 05
АД
а зачем в my_test_task столько комитов с мусором?
АД
могли подумать, да, много умеет, ну нужно будет долго учить культуре и ревью будут очень сложными
AA
стоит ребейз сделать?
AA
я чёт даж не подумал о коммит хистори, мне нужно было оформлять это как продакшн проект
AA
сделать ветки, пул реквесты и тп?
AA
я в целом таким занимаюсь, но чёт даж не подумал в тестовой таске это делать, зря наверное
AA
спс за замечание, запишу себе сребейзить, как настроение появится
АД
ну э, не выйдет думаю
АД
получатся некоторые комиты без тестов например
АД
и тимлид ставит себя на место главного аппрувера пулл реквеста, смотрит на все это и такой: Нет
АД
тоесть какие-то команды согласятся тебя взять сеньером, какие-то опустят до мидла потому что культуры нет
АД
и сквошем это не решить
VA
тоесть какие-то команды согласятся тебя взять сеньером, какие-то опустят до мидла потому что культуры нет
а в чём культура заключается и где про неё читать?
мол, коммит должен иметь единственную ответственность, включать в себя тесты, нормальный заголовок, что-то еще?
мол, коммит должен иметь единственную ответственность, включать в себя тесты, нормальный заголовок, что-то еще?
АД
а в чём культура заключается и где про неё читать?
мол, коммит должен иметь единственную ответственность, включать в себя тесты, нормальный заголовок, что-то еще?
мол, коммит должен иметь единственную ответственность, включать в себя тесты, нормальный заголовок, что-то еще?
не комит, пулл реквест
про комиты ко мне не выдвигали таких требований, и я сам не вижу в этом смысла
про комиты ко мне не выдвигали таких требований, и я сам не вижу в этом смысла
АД
может быть комиит который убирает binding.pry - не зазорно, но он должен быть не в мастер ветке 🙂
АД
в мастер кладутся сквошнутые фичи/фиксы/ватевар
VA
окей, я имел ввиду коммит = сквошнутый пул реквест
АД
думаю да, я если требования еще вспомню напишу, но вроде как основное ты захватил