[질문] 깃허브 커밋 메시지 관련해서 다시 여쭙습니다 _ _)

[익명(222.236)]

1. 변수명 상수명 수정 - style

2. 사용하지 않는 import문 제거 - ??

3. 숫자들 상수로 바꿔주기 - refactor

   (맞추면 +1, 틀리면 -1 일 때 +=1, -=1 하던거를 += CORRECT_SCORE, -= WRONG_SCORE

이렇게 쓰면 되나요? 2번째는 chore 같은데 이걸 많이 안쓰는 것 같아 형님들이라면 뭘 사용하시는지 여쭙습니다 _ _)

[익명(119.197)]

Cleanup imports Extract to const

[익명(222.236)]

오 표현 좋네요 한글로 잘 바꿔서 써봐야겠습니다 감사합니다 _ _)

[익명(119.197)]

그냥 직접적으로 쓰는게 읽는 사람입장에서 낫지 않냐 Rename 같은거

[익명(222.236)]

그렇긴 한데 앵귤러 컨벤션에 없더라구요 ㅠ refacotr: import문 정리, 상수 추출 style : 변수명 변경 이런 식으로 쓰려구요 ..

[raker(rakerdam)]

refactor

[익명(222.236)]

역시 1번 style, 2,3번 refactor 쓰는게 맞겠죠 ?!

[익명(59.24)]

아니 걍 refactor 하나로 퉁치면됨

[익명(59.24)]

상세 분리하고 싶으면 refactor some module 이런식

[익명(222.236)]

저기 위에 3가지 다 refactor로 사용하시는 편인가요?

[익명(59.24)]

목적이 있으면 그걸적겠지 없으면 리팩터링아니면 머임

[익명(222.236)]

목적이 변수명 변경인데 앵귤러 보니까 refactor: 변수명 변경 이런식으로 적더라구요 ㅠㅠ

[black7375(221.168)]

1. Refactor: Rename - A to B 2. Refactor: Imports - unused 3. Refactor: Const - Number

[black7375(221.168)]

난 이런식으로 할 듯

[익명(222.236)]

아 그전에 커밋메시지를 한글로 이미 날려놔서 ㅠㅠ.. 감사합니다 !! 변수명 변경도 refactor를 사용하시는군요

[black7375(221.168)]

인터페이스에 영향이 있으면 fix써도 되고, 없다면 전 refactor로 취급합니다. 사람마다 기준이 조금씩 다를수는 있지만..

[익명(222.236)]

인터페이스가 전체적인 로직을 말씀하시는건가요?

[익명(222.236)]

혹시 int a; a = 10; 이라는 소스를 int a = 10; 으로 정리한다면 refactor: 소스 정리 이런식으로 하는게 맞는걸까요?

[black7375(221.168)]

외부 프로그램이나 모듈에 영향을 미칠수 있는 변경이요. 쉽게말해서 export하여 API로 쓰이는것들?

[익명(222.236)]

감사합니다 그럼 혹시 코드 정리는 뭘 사용하시나요? if문 3개이던거를 1개로 합친다던가, 선언하고 초기화 하는거를 동시에 한다던지 간단한 코드 정리들은 refactor가 맞나요 ?

[black7375(221.168)]

작동하는 로직상의 변경이 없으면 리팩터라고 취급할수 있을것 같고요, 리팩터링시에 동작이 변경될 수 있으니 유닛테스트를 할 수 있으면 쓰시는걸 권장드립니다

[익명(222.236)]

테스트케이스만 돌리면서 확인하고 있었는데 유닛테스트라는 것을 알아봐야겠네요 감사합니다..

[익명(124.48)]

chore:

[익명(222.236)]

감사하빈다 ~!~!

[C#(cpppp)]

솔직히 그런건 바꾼다고 문제생기는것도 아니고 잘 신경안쓰니 대충 뭉뚱그려도 됨