-
Notifications
You must be signed in to change notification settings - Fork 9
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
Consistent naming convention #53
Comments
Yeah, this ended up inconsistent. If someone puts together a PR, I'd be fine with just changing everything to |
Seconded |
To be clear, I actually meant the use of |
Just doing the function names has the nice advantage of being backwards compatible, so I'm fine with that option as well. I had interpreted your request as meaning everything, including the associated type family, but it would be fine to leave it as is since users do not often need to type it out. |
There are some inconsistent function names:
Mut
vsMutable
I don't really have an opinion on which is better, but I slightly prefer
Mut
, since it's shorter.foldMap
vsfoldlMap
I don't think it makes much sense to have
l
orr
infoldMap
-like functions, since their point is that the order doesn't matter.I think we should decide on a convention and then go with that, creating aliases and deprecating the old versions.
The text was updated successfully, but these errors were encountered: