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

feat: getChoseng을 utils에서 별도 함수로 분리합니다. #192

Merged
merged 5 commits into from
Jul 23, 2024

Conversation

okinawaa
Copy link
Member

Overview

#158

  1. getChoseong은 원자적인 함수이므로, 독립적인 함수로 위치하는게 자연스럽다고 생각했습니다.
  2. 과거에 잘못사용되던 getChosung을 제거합니다

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.

@okinawaa okinawaa requested a review from raon0211 as a code owner July 21, 2024 07:14
Copy link

changeset-bot bot commented Jul 21, 2024

⚠️ No Changeset found

Latest commit: 21f67f9

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

Copy link

vercel bot commented Jul 21, 2024

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

Name Status Preview Comments Updated (UTC)
es-hangul ❌ Failed (Inspect) Jul 21, 2024 8:00am

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.02%. Comparing base (fec20e8) to head (21f67f9).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##               v2     #192      +/-   ##
==========================================
+ Coverage   95.05%   96.02%   +0.97%     
==========================================
  Files          14       15       +1     
  Lines         283      277       -6     
  Branches       67       67              
==========================================
- Hits          269      266       -3     
+ Misses         13       10       -3     
  Partials        1        1              

@okinawaa okinawaa merged commit f115c54 into v2 Jul 23, 2024
9 of 10 checks passed
okinawaa added a commit that referenced this pull request Aug 4, 2024
* getChoseong분리

* write test code

* getChoseong import

* remove useless import statemenet

* remove unused file
okinawaa added a commit that referenced this pull request Aug 4, 2024
* getChoseong분리

* write test code

* getChoseong import

* remove useless import statemenet

* remove unused file
okinawaa added a commit that referenced this pull request Aug 5, 2024
* feat: 문자열에서 한글을 추출해주는 extractHangul 함수를 제거합니다 (#185)

* remove extrachHangul

* Create cyan-tigers-sneeze.md

---------

Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com>

* feat: disassembleHangul, disassemble, disassembleHangulToGroup 함수에서 hangul이라는 글자를 제거합니다 (#184)

* dissemble관련 메서드에서 hangul이름을 제거합니다

* 누락된 부분 수정

* resolve conflit

* Create late-beers-hang.md

* diassembleHangul to diassemble

---------

Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com>

* remove hangulIncludes (#188)

* feat: 한글의 두음을 반환해주는 acronymizeHangul 함수를 제거합니다. (#180)

* remove acronymize

* Create weak-walls-sniff.md

---------

Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com>

* 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: 서동휘 <8275026+suhdonghwi@users.noreply.github.com>

* fix: remove 한글

* fix: amountToMoneyCurrency로 함수명 변경

* fix: amountToHangul로 함수명 수정

* fix: conflict

* fix: filepath

* fix: import 방식 변경 및 CHANGELOG restore

* Create kind-birds-provide.md

---------

Co-authored-by: 박찬혁 <pgg6713@gmail.com>
Co-authored-by: Jonghyeon Ko <manudeli.ko@gmail.com>
Co-authored-by: 서동휘 <8275026+suhdonghwi@users.noreply.github.com>
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.

2 participants