[sprint review] 2주차_2023_07_21

[hemoi]

1. 신규 organization 생성, repository url 변경으로 인한 url 변경

https://github.com/shinhancard/dev-portal

2. workflow 설명
   https://github.com/shinhancard/dev-portal/wiki/workflow(github-flow)

• 모든 것은 이슈로 이슈별 브랜치를 만들어서 병합하기

• pipeline을 잘 구축하면 main으로만 보낸다면 문제 없이 병합되었다는 것을 확인할 수 있음

• rule: squash merge -> merge 하기 전 기존에 개발했던 것을 합쳐서 반영

• 업로드를 현행화 하기 위해서 푸시하는 것이지, 결국 다른 사람이 보는 것은 차이만을 보기 때문임

• 서버에서 테스트 되야하는 것은 별도로 deploy.yml에 feature/[name] 를 추가해야 함

• close # 를 pull request 에 추가한다면, 머지되면 close 된다

3. project init 리뷰

• babeling : 브라우저 호환성을 맞추기 위한

• nohoist 프로젝트 구조를 단순화

• 스토리북에도 만들고, 양쪽에도 만들면 불필요한 구조가 두개가 된다

• 스토리북도 별도로 package가 있기에 (디자인 작업을 위한 툴이다보니) 별도로 관리해야 함

• app, plugins 폴더가 있는데, 추후 dependency를 깔끔하게 유지하기 위해서 plugins에서 되도록이면 개발

• 추후 민준님이 prettier 다시 수정

4. 인환님 : postgresql db install

• 개발할때는 postgresql 붙을 필요는 없고 템플릿 등만 나중에 migration해서 반입하면 될 듯
• pgAdmin 추후 공유하기로

5. 원진님 : eks ci/ cd

• github 이 public이다보니 사내와 차이가 있었음
• flux로 배포후 당겨가도록 -> 사용하지 않는다면 쿠버네티스에 접속하기 위해서 configuration file을 cd 서버에 넣음 (만료가 없어 보안 별로)
• 보안성을 보완하기 위해서 CSP에서 토큰을 별도로 두고 kubeconfig로 업데이트하는 방식을 사용
• 이를 배제하기 위해서 클러스터에 flux를 설치하고, gitlab등에서 access token을 받아오는 것
• helm repository를 별도로 두기

6. 범수님 중현님 : component

• 백스테이지

7. 민준님 : prettier

• custom plugin을 별도로 두는 것이 좋을 것 같음
• app 안에 있는 것은 spotify를 그대로 사용하는 것이 좋음. 왜냐하면 수정되면 너무 많은 변경이 일어날 것 같음
• husky : ruleset 안맞으면 커밋이 안되는것 사용? -> 사용하지는 않고 vscode를 그대로 사용하는 것으로