[skiende74]

Overview

combineVowels의 리턴타입을 제네릭을 통해 type-narrowing 하도록 개선합니다.

PR Checklist

• I read and included theses actions below

1. I have read the Contributing Guide
2. I have written documents and tests, if needed.

[changeset-bot]

🦋 Changeset detected

Latest commit: 110460e

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
es-hangul	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 6, 2025 6:22am

[po4tion]

좋은 작업해주셔서 감사합니다! 혹시 comnbineVowels 로직의 가독성을 지금보다 더 향상시킬 수 있는 방법이 있을까요?

[skiende74]

@po4tion

좋은 작업해주셔서 감사합니다! 혹시 comnbineVowels 로직의 가독성을 지금보다 더 향상시킬 수 있는 방법이 있을까요?

type Invert<T extends Record<string, string>> = { [K in keyof T as T[K]]: K };
type CombineVowel = Invert<typeof DISASSEMBLED_VOWELS_BY_VOWEL>;

기존의 위 두 줄을

type Obj = typeof DISASSEMBLED_VOWELS_BY_VOWEL;
type CombineVowel = { [K in keyof Obj as Obj[K]]: K };

이렇게 고치면 조금 더 짧아질 것 같긴 합니다.

[po4tion]

@skiende74 님 좋은 의견 감사합니다!
@okinawaa 님 확인 부탁드립니다!

[okinawaa]

작업 감사합니다!