Skip to content

feat: Added fromatting for Credit Card Number - #139

Merged
Samarth2190 merged 3 commits into
Samarth2190:mainfrom
Farkhanda-Dalal:feature/creditCard
Oct 28, 2025
Merged

feat: Added fromatting for Credit Card Number#139
Samarth2190 merged 3 commits into
Samarth2190:mainfrom
Farkhanda-Dalal:feature/creditCard

Conversation

@Farkhanda-Dalal

Copy link
Copy Markdown
Contributor

Closes:
Closes issue #114

Changes

  • Added src/formatting/creditCard.ts file
  • Added src/test/formatting/creditCard.test.ts file
  • Modified src/formatting/index.ts
  • Modified README.md
Screenshot 2025-10-25 234354

@Samarth2190 Samarth2190 left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Issue fixed. Merging request.

Comment thread README.md Outdated

| Parameter | Type | Default | Description |
| ---------- | ------ | -------- | ------------------------------------------------------------------- |
| cardNumber | string | required | The credit card number to format. Can include non-digit characters. |

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Suggested change
| cardNumber | string | required | The credit card number to format. Can include non-digit characters. |
| cardNumber | string | required | The credit card number to format. Cannot include non-digit characters. |

I think it should be "Cannot include non-digit characters." as credit card number is digit only.

@Samarth2190
Samarth2190 merged commit f1d5ae9 into Samarth2190:main Oct 28, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants