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

toTitle helper is kinda confusing #32

Open
tylersticka opened this issue May 6, 2016 · 2 comments
Open

toTitle helper is kinda confusing #32

tylersticka opened this issue May 6, 2016 · 2 comments

Comments

@tylersticka
Copy link
Member

⚠️ Note: I might just be a dummy.

But I never remember that toTitle only strips off numeric prefixes and nothing else.

For some reason I always assume it will do this. Which isn't even a common need now that there are capitalize helpers.

Now that Drizzle doesn't rely on filename prefixes, it could be that we don't even need this helper. ¯_(ツ)_/¯

@erikjung
Copy link
Contributor

erikjung commented May 6, 2016

I wouldn't lose any sleep over removing it.

@spaceninja
Copy link
Member

@tylersticka agreed that's a confusing name. Sounds like the helper isn't needed anymore. Can you think of any reason we shouldn't just remove it and bump the NPM package version?

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

No branches or pull requests

3 participants