Skip to content

Cleanup reification of primitive liftables #7268

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

Merged

Conversation

nicolasstucki
Copy link
Contributor

No description provided.

@nicolasstucki nicolasstucki force-pushed the do-not-hardcode-primitive-liftables branch from f6bd430 to 77415fe Compare September 19, 2019 15:21
@nicolasstucki nicolasstucki requested review from liufengyun and removed request for liufengyun September 19, 2019 15:22
@nicolasstucki nicolasstucki force-pushed the do-not-hardcode-primitive-liftables branch from 244feb9 to 8b91123 Compare September 19, 2019 15:33
@nicolasstucki nicolasstucki marked this pull request as ready for review September 19, 2019 16:39
Copy link
Member

@smarter smarter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, why is doing an implicit search better than hardcoding the primitives ?

@nicolasstucki
Copy link
Contributor Author

It was mostly to be able to rename them easily. I will remove the implicit search.

@nicolasstucki nicolasstucki changed the title Do not hardcode primitive liftables Cleanup reification of primitive liftables Sep 24, 2019
@nicolasstucki nicolasstucki force-pushed the do-not-hardcode-primitive-liftables branch from 4adfcb3 to 186feba Compare September 24, 2019 09:21
@nicolasstucki nicolasstucki merged commit 09b45d3 into scala:master Sep 30, 2019
@nicolasstucki nicolasstucki deleted the do-not-hardcode-primitive-liftables branch September 30, 2019 13:35
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

Successfully merging this pull request may close these issues.

2 participants