당장 손절해야 할 동료 top 5

[익명(218.235)]

1. 변수명 a, b, ...로 짓는 개발자

2. ts에서 any 타입 쓰는 개발자

3. git commit 뭉텅이로 날려서 나중에 복구하라고하면 소스 뒤지는 개발자

4. 모듈화 안하고 작업 두번하는 개발자

5. 추상화 안해서 지도 이해안돼서 지가 짠 코드 디버깅 못하는 개발자

속터짐 ㅇㅇ

[익명(211.209)]

1번 2번은 약간 게으른타입임. 개발할때는 그렇게 할 수 있지만 푸쉬올릴 때는 다시 다 고쳐놔야지. 3,4,5는 상황마다 다름. 모듈화, 추상화는 함부러 하지 않는게 낫는 경우가 많음. git commit은 잘 나누는것도 중요하지만 리뷰가 활성화 되어있는곳에서는 하나로 뭉치고 리뷰어가 편하게 보게 하는게 남.

[익명(118.235)]

커밋 하나로 뭉쳐서 올리고 일부 개발 취소하라그러면 주석처리할 곳 눈으로 찾는 모지리들을 많이 봐서그래. feature 단위로 올려놓으면 문맥이 눈에 보이잖아. 되돌릴때 커밋revert만 하면되는걸 꼭 손으로 주석처리하고있더라고. 내 생각엔 리뷰어들도 feature 단위로 보는게 더 편할것같은데? 섞여있으면 코드의 의도가 안보이니까

[익명(211.209)]

너랑 나랑 같은말인데 PR을 feature단위로 올리고, 그 feature단위로 커밋을 하나로 뭉쳐서 올리는게 좋음. PR안올릴거면 commit 나눠 올리는거 좋은데, 나는 revert를 신뢰하는 편은 아니라서 (side-effect없는 모듈이면 가능하지만) 히스토리 찾는 용이긴 함.

[익명(211.209)]

단순히 git의 diff만을 믿고 모듈들 하나씩 revert시키면 예측하지 못한 상황에서 터질 수 있거든 테스트코드 준비되어있으면 이야기는 다름.

[나트륨찡(natriummi)]

[익명(118.235)]

맞는 말임. 애초에 잘하는 사람이면 revert만해도 코드는 물론 커밋도 모듈화 잘돼있어서 문제없는데 못하는사람들은 애매하게 섞여있어서 revert 쓰면 큰일남. 흔히누구나쓰는 commit, push, pull말고 git 다른 중/고급 기능들 reset, revert, rebase같은 것들은 그런 사람들은 아예 모르는게 약임ㅋㅋㅋ

[나트륨찡(natriummi)]