Skip to content

feat: add multiply util function#26

Merged
prgmr99 merged 2 commits intomainfrom
feature/utils-workspace-yeom
Sep 1, 2025
Merged

feat: add multiply util function#26
prgmr99 merged 2 commits intomainfrom
feature/utils-workspace-yeom

Conversation

@prgmr99
Copy link
Copy Markdown
Member

@prgmr99 prgmr99 commented Sep 1, 2025

Description

multiply 유틸함수를 추가합니다.
- 이 함수는 여러 개의 숫자 인자를 받아 모두 곱한 결과를 반환합니다.
- reduce를 사용하여 구현했으며, 인자가 없을 때의 엣지 케이스를 처리합니다.
- NaN이 들어오는 경우도 함께 처리합니다.

close #25

@prgmr99 prgmr99 requested a review from klmhyeonwoo September 1, 2025 10:46
@prgmr99 prgmr99 self-assigned this Sep 1, 2025
@prgmr99 prgmr99 marked this pull request as draft September 1, 2025 10:46
@klmhyeonwoo
Copy link
Copy Markdown
Member

klmhyeonwoo commented Sep 1, 2025

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 62.06% 90 / 145
🔵 Statements 62.06% 90 / 145
🔵 Functions 93.33% 14 / 15
🔵 Branches 95.83% 46 / 48
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
package/numberUtil/multiply/index.ts 100% 100% 100% 100%
Generated in workflow #40 for commit 4d641ff by the Vitest Coverage Report Action

@prgmr99 prgmr99 marked this pull request as ready for review September 1, 2025 12:17
Copy link
Copy Markdown
Member

@klmhyeonwoo klmhyeonwoo left a comment

Choose a reason for hiding this comment

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

굿입니다요!!

@prgmr99 prgmr99 merged commit 1775b56 into main Sep 1, 2025
1 check 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

Development

Successfully merging this pull request may close these issues.

numberUtil > multiply()

2 participants