toss / es-hangul

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

fix: 굳이 모두 풀었다가 합치지 않고 마지막 글자만 적용합니다. #114

Closed crucifyer closed 4 months ago

crucifyer commented 5 months ago

Overview

연습삼아 pr 해봤습니다. 동작을 간소화 할 만한 부분이 조금 보이네요. (overview 내용만 고쳤는데, 주석이었네요.)

PR Checklist

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

⚠️ No Changeset found

Latest commit: e206b72680faf2f5fe4dcd4b10ea172a41b9be72

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

vercel[bot] commented 5 months ago

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

Name Status Preview Comments Updated (UTC)
es-hangul ❌ Failed (Inspect) Jun 7, 2024 9:59am
okinawaa commented 5 months ago

기여 감사합니다! PR Description을 적절히 채워주시면 감사하겠습니다~! 또한 코드 내에 주석또한 의도된 것인지 궁금해요!

crucifyer commented 5 months ago

overview 에 쓴 글이 주석상태였네요. 수정했습니다.

하나 해 보고 어떻게 처리되는지 본 후 나머지 올리려 했는데, commit 한 것들이 다 올라가버렸네요. pr 내 모든 코드의 목적은 동작의 효율화 이고, 기존 코드를 삭제하지 않고 주석하는것은 그냥 습관적인 것으로 코드비교가 잘 되는 github 에는 안 맞을 수도 있을 것 같습니다.

okinawaa commented 4 months ago

기여 감사합니다! 깔끔한 코드 관리를 위해, 필요하지 않은 코드는 주석이 아닌 삭제를 하는 것이 어떨까요? 또한 어떠한 부분에서 동작의 효율화가 일어났는지 서술해주시면 더 좋을 것 같아요!

crucifyer commented 4 months ago

commit 에 적은 메세지만으로 설명은 충분하다고 생각되는데, 가독성이 떨어지나보네요. 점점 지저분해 지기도 하고, 닫고 다시 도전해보겠습니다.