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

Just a general question. Does having more commits on PR

create bad impression?

22 ответов

29 просмотров

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 | PM me for Paid Support
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

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

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

Добрый вечер. Есть вопрос, а может и предложение. Был у меня диалог в другой группе о делфи и я задался вопросом: "А нельзя ли в делфи цвет //коментария и {комментария} сде...
Kraszx
24
Мдя, прикол, боевая сборка запускается (именно под отладчиком) после F9 примерно полторы минуты (97 секунд если быть точным). Начал копать - проблема детектится сразу - зависа...
Александр (Rouse_) Багель
38
Всем привет! Подскажи, пожалуйста, как передать в TComboBox сразу значение и id записи. На Delphi я делал так: ComboBox1.Items.AddObject('Какое-то значение', Pointer(id запис...
Евгений
10
я так понимаю, я так подозреваю, что создание такого плагина для человека, кто умеет писать плагины для делфи потребует минут 5-10 времени. но это мое подозрение. хотелось бы ...
Kraszx
7
Товарищи, кто работа с iphelper? Или может я в самой логике ошибки фигачу, не пойму.... var ifTable : PMIB_IFTABLE; size, corSize: DWORD; Buffer ...
Warfarellen
4
Здравствуйте, вопрос по структурам данных. Были у вас случаи, когда пришлось писать деревья или двунаправленные списки?
/ /
50
Коллеги, добрый вечер. Создаю коллекцию от TFPGMap, ключ - перечисление, значение - целое. Нужно отсортировать коллекцию по значению. Как это можно сделать?
Kirill Filippenok
11
Привет, такой вопросик появился кажется ли вам что Rust слишком сложный/строгий для высокоуровневого программирования и слишком "безопасный"/строгий для низкоуровневого?
Крокант
10
Скажи а ты когда этот канал создавал ты уже дельфи не любил, или это со временем пришло?
Роман Лях (rgreat)
18
Всем привет! Использую кастомное модальное диалоговое окошко, все по классике - mrOK, mrCancel как ModalResult. Однако есть нюанс - в главной форме есть универсальный обработч...
Олег Гранишевский
20
Карта сайта