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

Change 'identity' to 'id' back #129

Closed
chshersh opened this issue Feb 2, 2018 · 4 comments
Closed

Change 'identity' to 'id' back #129

chshersh opened this issue Feb 2, 2018 · 4 comments

Comments

@chshersh
Copy link
Contributor

chshersh commented Feb 2, 2018

This is more user friendly.

@chshersh chshersh added this to the v1.2.0: Convenience milestone Feb 3, 2018
@gromakovsky
Copy link
Member

Sometimes I even see import Data.Function (id).

@volhovm
Copy link
Contributor

volhovm commented Feb 7, 2018

I'd leave both of them to preserve backward compatibility. The original point was "id is a good name for some variable", but as far as i see, it's a super rare usecase.

@johnchandlerburnham
Copy link
Contributor

id :: a -> a is basically an idiom. If used as a variable, it's sort-of like an implicit overloading that might confuse people. There are other options for variable naming like idN or idT that are almost as good

@chshersh
Copy link
Contributor Author

Let's export id but keep identity as well for compatibility.

vrom911 added a commit that referenced this issue Feb 19, 2018
chshersh added a commit that referenced this issue Feb 19, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants