-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
feat: Add project_replace #43
Conversation
Thanks! I will review it in detail next week, but overall it looks great! pin-project has 4 important tests on |
3177cf0
to
e132c2c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me. r=me after nit addressed
bors d+
✌️ Marwes can now approve this pull request. To approve and merge a pull request, simply reply with |
490ed06
to
34b42bc
Compare
@Marwes Could you rebase and update the output of expandtest ( |
To mirror the same method in [pin-project](https://docs.rs/pin-project/1.0.3/pin_project/). Currently this has to been specified explicitly via an attribute as it otherwise breaks on `!Sized` types.
34b42bc
to
d17b34b
Compare
d17b34b
to
cf30fc4
Compare
bors r+ |
Build succeeded: |
Published in 0.2.4. |
To mirror the same method in pin-project.
Currently this has to been specified explicitly via an attribute as it
otherwise breaks on
!Sized
types.Could use some cleanup of the generated code and some tests around drop order (if possible) but other than that this did not seems to be as hard as I thought.
Closes #42