[Git] Pull Request 수정 방법

작성 계기

빨리 코드를 push하고 집에가고 싶어서 제대로 확인을 안하고 push를 하여
DB 접속 정보를 pr에 올렸다.

자고 일어나서 코드리뷰를 보고 급하게 수정을 하려고 했지만 많은 commit 때문에
수정을 하기가 굉장히 난감했다.

이를 해결한 과정을 기록하고 싶어서 글을 작성하게 되었다.

해결방법

• application.properties로 예를 들겠다

1. 가장 먼저 해당 코드를 어떤 시점에서 commit을 했는지 찾아야 한다
   

   수정을 하고도 적용되지 않아 확인해보니까 초기 시점에서도 application.properties를 commit 했던 기록이 남아 있어서 변경사항이 적용되지 않았다.

   따라서 만약 해당 방법으로 해결을 하지 못한다면 이전 시점에서 해당 코드를 commit한 기록이 있는지 확인을 하길 바란다.
   

2. commit을 취소한다

   $ git reset HEAD^ // 가장 최근의 commit을 취소한다

   $ git reset [commitID] // 원하는 시점으로 워킹 디렉토리를 되돌린다

   

   1번에서 돌아갈 시점을 찾았으니까 해당 시점까지 commit을 취소한다.
   가까운 시점이면 처음 명령어를 반복해서 입력하면 된다.
   귀찮으면 commit id를 통해서 한번에 해당 시점으로 갈 수 있다.

   git log or git log --oneline으로 확인을 하면서 작업을 진행하는 것을 추천한다.

   나는 해당 시점으로 돌아가도 코드가 없어지지 않았지만 혹시 모르니 코드는 백업을 해두고 작업을 진행하는 것을 추천한다.

3. 다시 commit을 한다.
   해당 코드를 제외하고 다시 commit을 한다.

4. push를 한다.
   아래 명령어를 통해 push를 한다.
   -f를 붙이지 않으면 오류가 발생할 것이다.

   git push origin [branch] -f

이렇게 하고 pr에 들어가보면 코드가 수정된 것을 확인할 수 있다.

참조 : https://gmlwjd9405.github.io/2018/05/25/git-add-cancle.html

---

느낀점

이번에 이 과정을 겪고나서 push를 할때는 신중하게 해야 한다는 점을 깨달았다.
또한 프로젝트 초기에 gitignore를 설정을 잘 해야한다는 것을 알게되었다.

추가

• gitignore 파일에 추가 후 반영하기

gitignore를 사용하고 싶으면 기존에 버전관리 했던 내용을 삭제해줘야 한다.

git rm --cached 추가할 파일 경로// C:/git/Back-End/src/main/resources/application.properties  

위의 명령어를 실행 후 아래처럼 변경사항을 push하면

git add .
git commit -m "설명"
git push

gitignore가 정상작동되고 application.properties가 업로드 되지 않는다.

참조 : https://gmlwjd9405.github.io/2018/05/17/git-delete-incorrect-files.html