search

toss / es-hangul

A modern JavaScript library for handling Hangul characters.
https://es-hangul.slash.page/
MIT License
1.24k stars 83 forks source link

feat: 문자열에서 한글을 추출해주는 extractHangul 함수를 제거합니다 #185

Closed okinawaa closed 1 month ago

okinawaa commented 1 month ago

Overview

https://github.com/toss/es-hangul/discussions/176 에서 논의한 결과 extrachHangul은 한글의 원리를 해결하는것이 아닌, 자바스크립트 일반 메서드로도 해결이 가능하므로 es-hangul에서 제공해주지 않아도 된다고 생각합니다.

isHangul 함수만 제공해서 [..."가a나b"].filter(isHangul).join('')처럼 사용하게 하는 것이 더 나은 인터페이스라고 생각해요.

PR Checklist

  1. I have read the Contributing Guide
  2. I have written documents and tests, if needed.
changeset-bot[bot] commented 1 month ago

🦋 Changeset detected

Latest commit: 0e5c2def39dbe215304919bb90c7a0e3d5980f90

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[bot] commented 1 month ago

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

Name Status Preview Comments Updated (UTC)
es-hangul ❌ Failed (Inspect) Jul 14, 2024 7:25am
codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.15%. Comparing base (c96e34e) to head (0e5c2de).

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/toss/es-hangul/pull/185/graphs/tree.svg?width=650&height=150&src=pr&token=My9jTW6bSr&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=toss)](https://app.codecov.io/gh/toss/es-hangul/pull/185?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=toss) ```diff @@ 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 ```
manudeli commented 1 month ago

⚠️ No Changeset found

Latest commit: cd92895

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

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

Changeset 추가가 필요합니다

okinawaa commented 1 month ago

@manudeli 님 v2로 올라갈때, v2 => main 으로 진행하는 PR에서만 changeset추가를 하면 되는게 아니라, feature => v2로 올라가는 각 PR마다 changeset이 필요한가용!?

okinawaa commented 1 month ago

130 에서 구현해주신 @Collection50 님의 생각은 어떠신지 궁금해요!

manudeli commented 1 month ago

@manudeli 님 v2로 올라갈때, v2 => main 으로 진행하는 PR에서만 changeset추가를 하면 되는게 아니라, feature => v2로 올라가는 각 PR마다 changeset이 필요한가용!?

넵넵 각 pr의 변경점을 v2에서 취합해서 한 번에 v2 -> main에서 체인지로그가 나올거에요

그래서 모든 브레이킹 체인지에서 메이저를 올려야 맞아요. 메이저 체인지셋이 여러개여도 메이저버전 하나만 올라가요

okinawaa commented 1 month ago

아하~ 체인지로그를 쌓는다는 관점에서 각 PR마다 changet을 추가해야겠군요! 감사해요

Collection50 commented 1 month ago

https://github.com/toss/es-hangul/pull/130 에서 구현해주신 @Collection50 님의 생각은 어떠신지 궁금해요!

176 에서 나누신 이야기 중, 더 뾰족한 라이브러리가 되는 것에 완전히 공감합니다 !!