feat: add acronymize function to string utilities#1690
feat: add acronymize function to string utilities#1690arimotearipo wants to merge 1 commit intotoss:mainfrom
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull request overview
Adds a new acronymize helper to the src/string utilities module, along with a Vitest spec and a barrel export so consumers can import it from the package’s string entrypoint.
Changes:
- Introduce
acronymize(str)to build an acronym from the words parsed by the existingwords()utility. - Add Vitest coverage for common acronymization cases (whitespace, punctuation, numbers).
- Export
acronymizefromsrc/string/index.ts.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| src/string/index.ts | Exposes acronymize from the string barrel export. |
| src/string/acronymize.ts | Implements acronym generation using words(). |
| src/string/acronymize.spec.ts | Adds unit tests for the new acronymize utility. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
82d93a6 to
9727af1
Compare
40c9f30 to
f2a2ed7
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
f2a2ed7 to
e3021bb
Compare
e3021bb to
db6e7df
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.