Add people.json 데이터 검증 CI workflow 추가 - Githubissues

AUSG / 2022-homepage

2022년 AUSG 홈페이지 리뉴얼 프로젝트

https://ausg.me

The Unlicense

4 stars 1 forks source link

Add people.json 데이터 검증 CI workflow 추가 #63

Closed mokhs00 closed 1 year ago

mokhs00 commented 1 year ago

Summary

Close #50

수빈님이 만들어준 7기 온보딩 페이지를 통해 다수의 인원을 자발적으로 people.json에 데이터를 추가하는 시점이 되었고
이에 따라 자주 수정이 발생하며 휴먼 에러가 발생하기 딱 좋은 구조가 되었습니다.
그래서 people.json 데이터를 자동으로 검증하는 테스트를 추가하여 CI 레벨에서 이를 검증하고 리뷰어의 시간을 아껴주는 장치를 추가합니다.

Action Items

year 데이터를 추가하지 않으면 로직상 홈페이지에 노출되지 않는 경우가 있었습니다. 관련해서 People 데이터 타입을 SSOT 바라보도록 리팩토링이 필요해 보입니다.
일부 SSOT를 바라보는 게 이상적이지만 산발적으로 분산된 코드도 있습니다. 해당 코드도 정리가 필요해보입니다. (같은 TAB 데이터를 다루는 코드1, 코드2)
import 경로가 상대경로인 경우가 많습니다. ts-config와 import 경로를 적절하게 alias로 교체가 필요합니다.

관련해서 의견 주시면 감사합니다. 이 PR 작업 후 Issues에도 추가해둘게요! -> https://github.com/AUSG/2022-homepage/issues/64 추가 완료

잘 모르고 짠 거라 무한 훈수 감사합니다 🙏