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

docs: es-hangul의 빠름을 보여줄 수 있는 벤치마크 페이지를 만듭니다. #286

Merged
merged 2 commits into from
Nov 20, 2024

Conversation

okinawaa
Copy link
Member

@okinawaa okinawaa commented Nov 20, 2024

Overview

fix: #284

Screen.Recording.2024-11-20.at.1.07.49.PM.mov

이번 PR은 es-hangul의 벤치마크 결과를 보다 시각적이고 정보를 효과적으로 전달할 수 있도록 개선하는 내용을 포함하고 있습니다. 업데이트된 벤치마크 차트를 통해 es-hangul이 다른 라이브러리와 비교했을 때 압도적인 성능 우위를 강조합니다.

  1. 벤치마크 차트
    • React를 사용해 es-hangul, auto-josa, josa의 성능 비교를 시각적으로 표현한 벤치마크 차트 추가.
    • 스크롤 시 차트 애니메이션으로 성능 차이를 강조.
  2. 언어 로컬라이제이션
    • 주요 UI 요소와 벤치마크 정보를 영어로 번역해 국제 사용자들도 이해할 수 있도록 개선.
  3. 네비게이션 업데이트
    • “더 자세한 벤치마크 결과 보기” 버튼을 클릭 시 GitHub 벤치마크 페이지가 새 탭에서 열리도록 수정.
  4. MDX 문서 추가
    • es-hangul의 장점과 성능을 효과적으로 전달하기 위한 MDX 파일 추가.
    • 벤치마크 결과 및 이미지를 포함하여 사용자들에게 신뢰감을 줄 수 있는 내용을 작성.

왜 중요한가요?

es-hangul은 높은 성능을 자랑하는 라이브러리로, 이를 개발자들에게 제대로 소구하는 것이 중요합니다. 이번 업데이트는 es-hangul의 신뢰성과 성능 우수성을 강조해 더 많은 개발자들이 이 라이브러리를 사용할 수 있도록 돕습니다.

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 November 20, 2024 04:08
Copy link

changeset-bot bot commented Nov 20, 2024

⚠️ No Changeset found

Latest commit: 72c43ce

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 Nov 20, 2024

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

Name Status Preview Comments Updated (UTC)
es-hangul ❌ Failed (Inspect) Nov 20, 2024 4:09am

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.53%. Comparing base (cd09392) to head (72c43ce).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #286   +/-   ##
=======================================
  Coverage   99.53%   99.53%           
=======================================
  Files          38       38           
  Lines         642      642           
  Branches      156      156           
=======================================
  Hits          639      639           
  Misses          3        3           
---- 🚨 Try these New Features:

Copy link
Collaborator

@po4tion po4tion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

정말 예쁘게 벤치마크 페이지가 탄생한 것 같아요!

);
};

const 천만 = 10000000;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

숫자 단위가 높을 때는 10_000_000으로 표현하는 것은 어떤가요?
좀 더 알아보기 편할 것 같다고 생각해요!
또한 정말 천만이라는 변수명에 알맞는지 빠르게 검증할 수 있을 것 같아요.

@okinawaa okinawaa merged commit 7d3f4b1 into main Nov 20, 2024
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants