feat: 문자열에서 한글을 추출해주는 extractHangul 함수를 제거합니다 #185
Conversation
🦋 Changeset detectedLatest commit: 0e5c2de 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 |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## v2 #185 +/- ##
==========================================
- Coverage 95.18% 95.15% -0.04%
==========================================
Files 17 16 -1
Lines 291 289 -2
Branches 67 67
==========================================
- Hits 277 275 -2
Misses 13 13
Partials 1 1 |
Changeset 추가가 필요합니다 |
|
@manudeli 님 v2로 올라갈때, |
|
#130 에서 구현해주신 @Collection50 님의 생각은 어떠신지 궁금해요! |
넵넵 각 pr의 변경점을 v2에서 취합해서 한 번에 v2 -> main에서 체인지로그가 나올거에요 그래서 모든 브레이킹 체인지에서 메이저를 올려야 맞아요. 메이저 체인지셋이 여러개여도 메이저버전 하나만 올라가요 |
|
아하~ 체인지로그를 쌓는다는 관점에서 각 PR마다 changet을 추가해야겠군요! 감사해요 |
#176 에서 나누신 이야기 중, 더 뾰족한 라이브러리가 되는 것에 완전히 공감합니다 !! |
* remove extrachHangul * Create cyan-tigers-sneeze.md --------- Co-authored-by: Jonghyeon Ko <[email protected]>
* remove extrachHangul * Create cyan-tigers-sneeze.md --------- Co-authored-by: Jonghyeon Ko <[email protected]>
* feat: 문자열에서 한글을 추출해주는 extractHangul 함수를 제거합니다 (#185) * remove extrachHangul * Create cyan-tigers-sneeze.md --------- Co-authored-by: Jonghyeon Ko <[email protected]> * feat: disassembleHangul, disassemble, disassembleHangulToGroup 함수에서 hangul이라는 글자를 제거합니다 (#184) * dissemble관련 메서드에서 hangul이름을 제거합니다 * 누락된 부분 수정 * resolve conflit * Create late-beers-hang.md * diassembleHangul to diassemble --------- Co-authored-by: Jonghyeon Ko <[email protected]> * remove hangulIncludes (#188) * feat: 한글의 두음을 반환해주는 acronymizeHangul 함수를 제거합니다. (#180) * remove acronymize * Create weak-walls-sniff.md --------- Co-authored-by: Jonghyeon Ko <[email protected]> * feat: getChoseng을 utils에서 별도 함수로 분리합니다. (#192) * getChoseong분리 * write test code * getChoseong import * remove useless import statemenet * remove unused file * feat: `canBeChoseong`, `canBeJungseong`, `canBeJongseong` 을 utils에서 별도 함수로 분리합니다. (#193) * canBe series를 별도 함수로 분리합니다 * add change set * feat: `hasBatchim` 을 utils에서 별도 함수로 분리합니다. (#195) * hasBatchim * resolve type error * add change set * hasProperty, hasValueInReadOnlyStringList를 internal folder 로 옮깁니다 * resolve test error * move with test code * feat: choseongIncludes함수를 제거합니다. (#197) * remove choseongIncludes Co-authored-by: 서동휘 <[email protected]> * fix: remove 한글 * fix: amountToMoneyCurrency로 함수명 변경 * fix: amountToHangul로 함수명 수정 * fix: conflict * fix: filepath * fix: import 방식 변경 및 CHANGELOG restore * Create kind-birds-provide.md --------- Co-authored-by: 박찬혁 <[email protected]> Co-authored-by: Jonghyeon Ko <[email protected]> Co-authored-by: 서동휘 <[email protected]>
Overview
#176 에서 논의한 결과
extrachHangul은 한글의 원리를 해결하는것이 아닌, 자바스크립트 일반 메서드로도 해결이 가능하므로 es-hangul에서 제공해주지 않아도 된다고 생각합니다.
PR Checklist