chore: remove .github/decisions directory#269
Conversation
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (2)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. 🚀 New features to boost your workflow:
|
3o14
left a comment
There was a problem hiding this comment.
기록의 목적은 이후 메인테이너가 되실 분들이 이전의 의사결정 근거들을 확인하실 수 있도록 맥락 아카이빙 용도에 가깝습니다. 제거보다는 남겨두는걸 희망해요. 위치는 상관없습니다.
아직까진 tokens 관련된 내용만 있어서 packages/tokens/docs/decisions/ 도 좋은 것 같네요.
강한 의견은 아니어서 우선 어프룹 해둘게요!
요약
.github/decisions/디렉토리 삭제 (token-architecture.md, token-naming-convention.md, v1-audit.md)질문
@3o14 원주님, 해당 파일들이
.github/decisions/에 위치한 게 의도적인 건가요?ADR(Architecture Decision Records) 성격의 문서인데,
.github/디렉토리는 보통 GitHub 메타 설정 파일 (workflows,ISSUE_TEMPLATE등)이 위치하는 곳이라 다소 어색하게 느껴졌습니다.관련 토큰 구현도 이미 완료된 것으로 보여 (
feat(tokens): add Style Dictionary v5 pipeline등) 삭제해도 무방하다고 판단했지만, 의도가 있었다면 말씀해 주세요!만약 문서 보존이 필요하다면
docs/하위로 이동하는 방향도 고려할 수 있을 것 같습니다.체크리스트
.github/decisions/내 파일 3개 삭제