T
Грустно вам :-(
Size: a a a
2019 January 28
ПФ
Грустно вам :-(
Ну так бывает. Это хорошая почва для того чтобы что-то улучшать. Но вот в моменте приходится бюрократию разводить, да
AV
А, скажем, если баг, внесённый в мастер после последнего релиза, можно и обойтись без тикета. Надо просто понимать, для чего нужны тикеты, тогда проблем нет
SB
В одном коммите с фиксом должен быть тесткейс, конечно. Если фикс внёс новый баг, фиксай заново, не ломая тот тесткейс.
Это лучше отвечает на вопрос "что это" но не всегда отвечает на вопрос "зачем это" и тем более "почему оно именно так". В условиях неидеального мира лучше всё же иметь связанный тикет.
T
Это лучше отвечает на вопрос "что это" но не всегда отвечает на вопрос "зачем это" и тем более "почему оно именно так". В условиях неидеального мира лучше всё же иметь связанный тикет.
И наивно надеяться, что тикет на что-то ответит :-D
SB
И наивно надеяться, что тикет на что-то ответит :-D
Из моей практики чаще да, чем нет. Ну и его номер всё-таки ещё одна зацепка для поиска в почте/скайпе/etc. )
TC
И если использовать тикеты для ведения релиз ноутов и диффов изменений, то внезапный фикс без тикета может в этом месте потеряться
T
И если использовать тикеты для ведения релиз ноутов и диффов изменений, то внезапный фикс без тикета может в этом месте потеряться
Зачем в релиз-нотес писать "исправлено исключение в новой фиче, которую вы сломанной никогда не видели, потому что раньше её не было"?
T
У нас релиз-нотес и так жирный
TC
Зачем в релиз-нотес писать "исправлено исключение в новой фиче, которую вы сломанной никогда не видели, потому что раньше её не было"?
Это скорее ко второму пункту про дифф - не показывать заказчику, но посмотреть самим внутри, если например несколько десятков компонент выдается
TC
По своей маленькой туда я тоже никому не пишу, если «мы случайно сами нашли NPE и починили»
Но тикет потом по крайней мере скажет, что нашли сами, а не пользователи принесли
Но тикет потом по крайней мере скажет, что нашли сами, а не пользователи принесли
T
Мы когда проперти-тесты ввели, наши сотни багов. Реально сотни. Если б на каждый тикет заводили, то подохли бы.
T
Нашли
TC
Ну, поэтому серебряной пули нет, и надо смотреть в каждом конкретном енве отдельно
AV
Зачем в релиз-нотес писать "исправлено исключение в новой фиче, которую вы сломанной никогда не видели, потому что раньше её не было"?
Или "пофикшен баг копипасты, который нечаянно появился во время фикса бага 2315"
I
Зачем в релиз-нотес писать "исправлено исключение в новой фиче, которую вы сломанной никогда не видели, потому что раньше её не было"?
Тогда это доделывание новой фичи и можно привязать коммит к таску с этой фичей
TC
Тогда это доделывание новой фичи и можно привязать коммит к таску с этой фичей
Да!
2019 January 29
T
Тогда это доделывание новой фичи и можно привязать коммит к таску с этой фичей
Ну вот в jdk на это бы новый тикет завели. Там один тикет - один коммит
ПФ
Ну вот в jdk на это бы новый тикет завели. Там один тикет - один коммит
И при этом у них нормально валидируются таски на предмет полезности м понятности
AT
Это прям радужная штука. Когда я работал в ЦНС мы так могли. В ламоде с гигантской вероятностью так не выйдет - слишком мало тестов
Самый простой и разумный способ внедрять тесты в старый код- писать их на все баги во время правки.