O
Size: a a a
2020 December 22
ДП
👍
краще туди, тут не канал для найму\хайрінга
AV
А что значит 1 коммит на задачу не хорошо?
ну вот как раз пишут про сквош, имхо иногда нужен черри пик, да и скрывать нечего мне, что я мог где-то подкосячить)
O
краще туди, тут не канал для найму\хайрінга
Спасибо, спамить не буду) Но если кто-то в поиске новых возможностей, жду в личку)
SC
Sergey Korol
@sergey_chepets только лучше сюда смотреть: https://github.com/sskorol/ts-test-decorators/blob/master/src/index.ts#L40-L59 Пока что отдельной репой. Есть в планах интегрировать в allure-js отдельным модулем. Но пока нет времени на это.
Спасибо. А я как раз недавно искал у тебя на гите DataSupplier для TS и не нашел)
SK
Как по мне если пр надо разбивать на коммиты чтобы легче ревьювить – то это уже слишком. Пр как по мне в идеале должен решать ровно одну задачу, которую можно выразить 1 коммитов в итоговом логе
Тут скорее речь о коммитах при фиксе комментов. Если сразу сквошить, то следующий раунд ревью приходится заново пересматривать весь PR. Потому лучше сквошить в самом конце, дабы в каждом раунде ревьюверам был виден точечный дифф.
DK
Sergey Korol
Тут скорее речь о коммитах при фиксе комментов. Если сразу сквошить, то следующий раунд ревью приходится заново пересматривать весь PR. Потому лучше сквошить в самом конце, дабы в каждом раунде ревьюверам был виден точечный дифф.
Сквошат обычно автоматически кнопочкой merge and squash, или вы не про это?
SK
Я о количестве коммитов. Во время ревью их может быть больше одного, и это нормально. Просто если каждый фикс сквошить (считая его решающим) - будет больно глазам ревьюверов.
NA
Пишу уже год минимум, ни разу не пригодидось)
бывает, мне приходилось с самого первого дня
просто не очень хорошо, если у вас это не принято на проекте как стандарт
просто не очень хорошо, если у вас это не принято на проекте как стандарт
NA
Все зависит от практик которые применяются в команде для работы с кодом. Если у вас требуется clean commit history, то rebase хорошо бы уметь делать. Ни в обиду сказано, но если за год не пригодилось - то вашей команде есть куда расти в области инженерных практик
++
AP
вопрос по puppeteer, чейто не пойму, почему я не могу перехтваить просто 'response' ивент? или как можно перехватить бади с респонса?
AP
если делать
page.on('response', response => console.info(`👉 Response: ${response.url()}`)); то все просто зависаетAP
а
page.on('request', request => console.info(`👉 Request: ${request.url()}`)); не могу дождаться респонсаAP
какойто замкнутый круг)
VG
ты не пробовал делать return response? 🙂
turn response? 🙂M
какойто замкнутый круг)
page.on('response', async(response) => {
const request = response.request();
if (request.url().includes('desiredrequest.json')){
const text = await response.text();
console.log(text);
}
})
const request = response.request();
if (request.url().includes('desiredrequest.json')){
const text = await response.text();
console.log(text);
}
})
AP
page.on('response', async(response) => {
const request = response.request();
if (request.url().includes('desiredrequest.json')){
const text = await response.text();
console.log(text);
}
})
const request = response.request();
if (request.url().includes('desiredrequest.json')){
const text = await response.text();
console.log(text);
}
})
так все просто зависет
AP
ты не пробовал делать return response? 🙂
turn response? 🙂не совсем понял
AP
page.on('response', async interceptedRequest => {
if (interceptedRequest.url().includes('blbla')) {
const body = await interceptedRequest.json();
console.log(body);
}
return interceptedRequest;
});AP
так тоже все зависате