chore: UI Kit 빌드 세팅 완료 #2
Conversation
evan-moon
requested review from
springkjw and
a team
and removed request for
springkjw
tsconfig.json
Outdated
| "strict": true, | ||
| "jsx": "react", | ||
| "moduleResolution": "node", | ||
| "declaration": true, | ||
| "forceConsistentCasingInFileNames": true, | ||
| "noFallthroughCasesInSwitch": true, |
There was a problem hiding this comment.
혹시 noFallthroughCasesInSwitch 관련 TypeError 발생하지 않나요?
관련해서 이슈도 있고 하는데 제 컴에서 계속 발생하고 있네요
There was a problem hiding this comment.
넵넵 저는 아직 경험해보지는 못했습니다...! 이 config가 안정적이지 않다면 빼도 상관없을 것 같아요. 엄청 중요한 녀석도 아니라서용
There was a problem hiding this comment.
요게 보니깐 최신 버전 react-scripts 에서 자동으로 tsconfig.json 파일 만들 때, suggested 옵션들이 조금 문제가 있네요. 나중에 요 문제 수정된 버전의 react-scripts 올리고 tsconfig.json 옵션도 바꾸면 될거 같아요.
There was a problem hiding this comment.
사실 react-script에 굳이 의존할 필요도 없기는 해서 그냥 eject해서 쓸만한 것만 남기고 저희 입맛대로 커스터마이징해도 무리 없을 것 같습니다.
| @@ -10,6 +10,7 @@ | |||
|
|
|||
| # production | |||
| /build | |||
There was a problem hiding this comment.
.vscode 폴더나 .idea 폴더 들도 같이 ignore 시키면 좋을거 같아요
| typescript({ | ||
| tsconfig: 'tsconfig.json', | ||
| }), | ||
| // babel({ |
There was a problem hiding this comment.
사실 요게 TDS 설정을 가져온 건데, 거기에는 바벨이 붙어있었거든용.
여기서는 굳이 필요없을 것 같기는 한데 TDS에서 바벨을 왜 붙힌건지 명확하게 파악이 안되어서 일단 주석으로만 처리해두었습니다. 도입 의도가 명확하게 파악되고 진짜 필요없다는 판단이 들면 제거하려고 합니다!
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
|
🎉 This PR is included in version 1.0.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
대충 빌드 와꾸만 잡아보았습니다.
Rollup추가*.d.ts로 타입스크립트에서 사용할 수 있게끔 셋업