Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Fix] 빌드 후 typescript 절대 경로 상대 경로로 변환하지 못하는 에러 해결 #43

Merged
merged 10 commits into from
Jun 9, 2024

Conversation

ghdtjgus76
Copy link
Collaborator

@ghdtjgus76 ghdtjgus76 commented Jun 8, 2024

🎉 변경 사항

기존 tsconfig에서 사용하던 절대 경로 설정이 있었는데, 빌드 후에도 상대 경로로 변환되는 것이 아니라 설정한 절대 경로 그대로 지정되어 타입을 불러오지 못했습니다.
첫 번째 사진은 빌드 전, 두 번째 사진은 빌드 후입니다.

image image

기존 설정으로 빌드할 때 @rollup/plugin-alias 라이브러리를 사용해서 절대 경로를 변환해줬습니다.
이는 hooks나 components 등 타입 선언 파일을 제외한 런타임에 돌아가는 파일들만 변환해줘서 타입 선언 파일에는 적용되지 않았습니다.

image

그래서 레퍼런스를 찾다가 아래 방법을 찾게 되었습니다.
가장 간단한 방법인 tsc-alias 방식을 사용했고, 문제가 생겼던 wow-ui, wow-icons에 둘 다 적용해놨습니다.
ezolenko/rollup-plugin-typescript2#201

해당 설정으로 빌드하면 아래와 같이 Chip 컴포넌트의 타입 선언 파일을 잘 불러오고 타입 관련 에러도 잘 뜨는 것을 볼 수 있습니다.
아이콘도 마찬가지로 타입 추론 잘 됩니다.

image image

🚩 관련 이슈

🙏 여기는 꼭 봐주세요!

Copy link
Contributor

github-actions bot commented Jun 8, 2024

packages/wow-ui/package.json Show resolved Hide resolved
packages/wow-ui/rollup.config.js Show resolved Hide resolved
Copy link
Collaborator

@SeieunYoo SeieunYoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

types 에서만 경로 인식이 안되는 거였군용,,
레퍼런스까지 감사합니다~~ LGTM 🚀🚀

apps/wow-docs/app/page.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

github-actions bot commented Jun 9, 2024

Copy link
Contributor

github-actions bot commented Jun 9, 2024

@ghdtjgus76 ghdtjgus76 merged commit 98a9b50 into main Jun 9, 2024
3 checks passed
@ghdtjgus76 ghdtjgus76 deleted the fix/tsconfig-alias-error branch June 9, 2024 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants