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.

Sign up for GitHub

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

Jump to bottom

Split by case should take , and space into account #92

Open
memochou1993 opened this issue Sep 22, 2024 · 0 comments · May be fixed by #98
Open

Split by case should take , and space into account #92

memochou1993 opened this issue Sep 22, 2024 · 0 comments · May be fixed by #98
Labels
bug Something isn't working

Comments

@memochou1993
Copy link

memochou1993 commented Sep 22, 2024 •
edited
Loading

Environment

  • scule: 1.3.0
  • Node: v20.13.0

Reproduction

titleCase('hello, world!')
// actual: Hello, world!
// expected: Hello, World!

titleCase('is_technical')
// actual: is Technical
// expected: Is Technical

Describe the bug

According to the documentation, titleCase is supposed to capitalize all words.

Even function words like "is," if they appear as the first word, should be capitalized.

Additional context

No response

Logs

No response
@memochou1993 memochou1993 added the bug Something isn't working label Sep 22, 2024
@memochou1993 memochou1993 changed the title About titleCase issue Issue with titleCase Sep 22, 2024
@pi0 pi0 changed the title Issue with titleCase Split by case should take , and space into account Sep 23, 2024
@nobkd nobkd mentioned this issue Nov 18, 2024
Open
@noootwo noootwo linked a pull request Nov 21, 2024 that will close this issue
Open
@Eazash Eazash mentioned this issue Jan 15, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Split by case should take space into account noootwo/scule
1 participant
@memochou1993