Dyskusja użytkownika Deykun

@serel dla pierwszych ssd to node_modules mialy znaczenie bo szybko zabijal ich zywotnosc. Aktualnie tam moze 2gb i nikt nie daje o to jebania

@wombatDaiquiri ja się z typem na końcu nie zgadzam, ale w sumie jego wyjaśnienie, że wszyscy implementujący swoje isNumber mają je w swoim kodzie i niech będzie duża paczka której podpaczki też to robią, to jakby wszystkie używały najpopularniejszego utilsa do tego to na koniec by było mniej kodu. To nie jest świat w którym chciałbym żyć, ale ta logika ma sens.

@wombatDaiquiri - ma to sens

this PR is to reduce network bandwidth because npm ships stuff as tarballs (and not source directly) and to reduce supply chain / dependency graph (it takes time to parse it + adds potential security risks)

@Deykun @koszotorobur jest dużo takich miejsc w kodzie, które można napisać lepiej. Ale podczas developmentu, żeby być efektywnym, potrzeba też „people skills” których tutaj ewidentnie zabrakło.

@wombatDaiquiri - umiejętności interpersonalne są niezwykle ważne - a im większy projekt i więcej ludzi jest w niego zaangażowanych tym są ważniejsze.

Niemniej jako owner projektu czasem trzeba podjąć decyzję wiedząc, że ona nie zadowoli części zaangażowanych w projekt ludzi.

Nie da się każdego zadowolić a umiejętność podjęcia decyzji i zakończenie jałowej dyskusji to cechy dobrego ownera.

W tym konkretnym przypadku faktycznie zabrakło miękkich umiejętności ale ostatecznie z technicznego punktu widzenia wprowadzona zmiana ma wystarczający sens i nie ma o co kopii kruszyć