Issue/22/account 관련 전반적인 리팩토링

[kaestro]

테스트를 실제로 돌리는 것이 불가능해(open api를 사용해야하는데 이와 관련한 세팅이 아직 되지 않음)

테스트 동작은 확인하지 못했음.

현재 작성한 형태로 프로젝트를 전반적으로 리팩토링하면 괜찮지 않을까 함

[hojin-kr]

좋아 실행되는거 확인되면 머지하자

[kaestro]

좋아 실행되는거 확인되면 머지하자

리뷰 안해줘서 ㅋㅋ

액션 통과했으면 실행 되는거야? 아니면 따로 돌려봐야돼?

[hojin-kr]

액션은 proto만 만드는거고, PR 올릴 때 테스트가 되고 올라와야할거 같다!

[hojin-kr]

그리고 구조가 바뀌면서 코드를 리뷰하면서 적용 여부에 대해서 검토하고 넣어야하는 내용으로 생각이되기도하고, 이런 경우에 대해서는 issue/22/account가 내가 작업하던 브랜치라고 보고, 별도 브랜치에서 issue/22/account로 PR을 하는 편이 좋을것 같아. 아니면 아직 issue/22/account가 main에 머지 되지 않았으니까, 별도 브랜치에서 main 으로 PR을 하는것도 방법일것 같고

[hojin-kr]

그리고 고 버전이 1.21로 쓰고 있었는데 1.22로 올라갔다 작업한 것들에 대해서

[kaestro]

ㅇㅇ 나도 네 코드가 효율적으로 동작한다고 생각하는데, 기존에 내가 코드를 짜는 형태들이랑은 달라서 나라면 이런식으로 짰을것 같은데...?라는 이야기 정도 하는거라서 한번 보고 스타일을 맞춰주면 고맙다는 제안 정도로 이해해주면 될 것 같아. 나는 struct에 들어갈 변수만 놓고 동작할 것을 따로 두는 것은 이제 이게 재활용해야하는 시점에서 struct가 있는 파일이 아니라 다른 main을 읽어야한다는건 이상하지 않을까라는 생각이 든 거거든

PR은 머지하고 싶다는 뜻이라기보다는 리뷰를 하려면 이런 수단을 써야하는 것 같아서 올린거고, 수정하라고 리젝 내려주면 충분할것 같애. 봐줘서 고마워

[kaestro]

go 버전 같은 경우는 내가 쓰고 있는게 22버전이라서 올려버린 모양이라 이건 내가 버추얼 환경 같은거로 이 프로젝트 진행할때 21을 쓰도록 하든지, 네가 22로 올리든지 해야할 것 같네. 어떻게 할래?