169 похожих чатов

Just a general question. Does having more commits on PR

create bad impression?

22 ответов

15 просмотров

I wouldn't care. I review the files and see the end results. If it works and doesn't screw the project over, it's a good PR, IMO

Adi- Автор вопроса
Agent Q | You PM me, You Pay
I wouldn't care. I review the files and see the en...

Having more commits in PR does it show that person is careless?

Garbage commits on default branch create bad impression

Adi
Having more commits in PR does it show that person...

As long as it's delivered and tested, I still don't care.

Adi- Автор вопроса
skittles
Define garbage commits

fix x oh no fix x finally fix x x took a long time to fix finally yayyyy x fixed

A​kshit
they do

idk why would anyone say they fixed X and again fix x

A​kshit
happens a lot lmao

they dont test their code?

skittles
they dont test their code?

idk but I have seen that happen

A​kshit
idk but I have seen that happen

impressive i thought i do garbage commits(commit half code in first commit then a whole ass full commmit of project)

skittles
impressive i thought i do garbage commits(commit h...

by half code i mean litterly half code in a file

Wojciech Pawlik
https://xkcd.com/1296/

well TBH when i do small changes to project(like renaming var or format/optimize imports) i write "Fixed few small stuff"

skittles
well TBH when i do small changes to project(like r...

I do a "misc" commit once in a while, fixing a bunch of stuff to small for a separate commit

Wojciech Pawlik
https://xkcd.com/1296/

true that, some of my commits for my private repos are like: fix stuff rollback stuff fix another stuff undo stuff haha

Похожие вопросы

Обсуждают сегодня

Скажите, можно ли как-то "переместить" динамический массив из одной переменной в другую? Скажем, переместить из TList<> в TArray<>. Именно переместить, а не скопировать. Если ...
Eugene Krasnikov (ᴊɪɴ x)
37
Вот еще криповенькая штука. uMain.pas(517,3) Warning: Case statement does not handle all possible cases И ЧО? 😂
Александр (Rouse_) Багель
20
комрады, че-та лыжы не едут var tmpFont: TFont; begin tmpFont:= TFont.Create; try case rgFontColor.ItemIndex of 0: tmpFont.Color:= clWindowText; 1: tmpFo...
Ed Doc
34
.model small .stack 100h .data a db 'Hello, World!', '$' ; исходная строка b db 20 dup(?) ; строка b с запасом на максимальную длину .code main: ...
Алексей -man
3
Интересно, нет ли какого-то способа получить из dll не адрес самой метки, а адрес со смещением?
The Bird of Hermes
54
вопрос, кого посмотреть в ютубе или где почитать про указатели чтобы раз и навсегда запомнить зачем они нужны и как правильно ими пользоваться? поделитесь хорошими ресурсами, ...
-
14
М-да. Почему бы просто со stringlist не работать?
Michael Longneck
23
Is there a digital way to cut the electricity from a usb in linux? It sounds weird, but it's exactly what I need to do. I tried to simulate the unplug/replug but is not the ...
Eduard Rivas
15
Редактор листа Excel, по сути двумерный массив ячеек. Ячейка - это экземпляр класса, у нее всякие свойства, методы. Проблема в том, что количество используемых строк и колоно...
Sergey Bodrov
2
Всем привет. Подскажите пожалуйста, как решить вопрос с подсветкой синтаксиса в vscode. Уже и разные плагины установил, и пробовал пошаманить в json settings, ничего не получ...
EEv9ENN 🤖
6
Карта сайта