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: Only named exports may use 'export type' #187

Merged
merged 1 commit into from
Aug 29, 2023
Merged

Conversation

TaehuiKim
Copy link
Contributor

@TaehuiKim TaehuiKim commented Aug 29, 2023

PR 의 종류는 어떤 것인가요?

  • 버그 수정
  • 새로운 기능
  • 리팩토링
  • 문서 수정
  • 워크플로우 수정

수정이 필요하게된 이유가 무엇인가요? (Jira 이슈가 있다면 링크를 연결해주세요)

microsoft/TypeScript#37238
export type을 해줄 때는 *를 사용할 수 없다.

=> microsoft/TypeScript#52217
ts 버전이 5.x대 버전으로 올라가면 해결될 문제로 보임.

무엇을 어떻게 변경했나요?

코드 변경을 이해하기 위한 배경지식이 필요하다면 설명 해주세요.

디펜던시 변경이 있나요?

어떻게 테스트 하셨나요?

코드의 실행결과를 볼 수 있는 로그나 이미지가 있다면 첨부해주세요.

@TaehuiKim TaehuiKim requested a review from a team August 29, 2023 03:47
@TaehuiKim TaehuiKim self-assigned this Aug 29, 2023
@day1-dev-adm day1-dev-adm added the bug Something isn't working label Aug 29, 2023
Copy link
Contributor

@jo-seongjun jo-seongjun left a comment

Choose a reason for hiding this comment

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

LGTM!!

export * from './type-util';
export type * from './type-util.type';
export { TypeUtil } from './type-util';
export type { NarrowableType, PickWithPartial, RequiredPartialProps, Nullable, Arrayable } from './type-util.type';

Choose a reason for hiding this comment

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

type 빼고 export * 로 써도 되지않나요?

Copy link
Contributor Author

@TaehuiKim TaehuiKim Aug 29, 2023

Choose a reason for hiding this comment

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

하지만 저친구들은 그저 타입이니77ㅏ... 흠
일단 이거 빌드 오류때문이라 병합하고 생각해볼게여

Copy link
Contributor Author

@TaehuiKim TaehuiKim Aug 29, 2023

Choose a reason for hiding this comment

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

microsoft/TypeScript#52217
더 보다 보니 나중 버전에는 지원하는 것 같아요
ts 버전이 5버전대로 올라가면 자연스럽게 해결 될 문제인걸로 보여요!

@TaehuiKim TaehuiKim merged commit cf20fff into main Aug 29, 2023
@TaehuiKim TaehuiKim deleted the fix/export_type branch August 29, 2023 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants