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

Release 0.4.0-alpha.6 #56

Merged
merged 1 commit into from
Sep 1, 2019
Merged

Release 0.4.0-alpha.6 #56

merged 1 commit into from
Sep 1, 2019

Conversation

taiki-e
Copy link
Owner

@taiki-e taiki-e commented Sep 1, 2019

No description provided.

@taiki-e taiki-e self-assigned this Sep 1, 2019
/// A struct generated by pin-project to provide an appropriate
/// `Unpin` implementation, this type's `Unpin` implementation
/// uses exactly the same conditions as the original type's
/// `Unpin` implementation.
Copy link
Owner Author

@taiki-e taiki-e Sep 1, 2019

Choose a reason for hiding this comment

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

I believe adding this will improve the generated documentation a little. (context: #53)

Structs

PublicStruct
__UnpinStructPublicStruct

A struct generated by pin-project to provide an appropriate Unpin implementation, this type's Unpin implementation uses exactly the same conditions as the original type's Unpin implementation.

Copy link
Owner Author

Choose a reason for hiding this comment

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

(Perhaps we can improve the documentation and naming more.)

Copy link
Owner Author

Choose a reason for hiding this comment

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

@Aaron1011 @Nemo157 Any thoughts?

@taiki-e
Copy link
Owner Author

taiki-e commented Sep 1, 2019

bors r+

bors bot added a commit that referenced this pull request Sep 1, 2019
56: Release 0.4.0-alpha.6 r=taiki-e a=taiki-e



Co-authored-by: Taiki Endo <te316e89@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 1, 2019

Build succeeded

  • taiki-e.pin-project

@bors bors bot merged commit f653147 into master Sep 1, 2019
@taiki-e taiki-e deleted the next-version branch September 1, 2019 09:48
bors bot added a commit that referenced this pull request Sep 3, 2019
62: Improve document of UnpinStruct r=Aaron1011 a=taiki-e

Improve document and add a way to opt-out mentioned in #53 (comment).

The reason for needing support for ways other than `-Zallow-features` is that it also affects the use of other dependency unstable features.

Related:  #54 #56 (comment)

Generated document:

<img width="1001" alt="struct-1" src="https://user-images.githubusercontent.com/43724913/64136451-95494280-ce2c-11e9-81ce-6a02c315332f.png">
<img width="1015" alt="struct-2" src="https://user-images.githubusercontent.com/43724913/64136462-a6924f00-ce2c-11e9-9d1c-3cdca2d9b158.png">

Ways to opt-out of this:

https://github.com/taiki-e/pin-project/blob/5f7c6e35c2a9f3dfbc01a63e6206875d1bfb986b/pin-project-internal/build.rs#L21-L28

Co-authored-by: Taiki Endo <te316e89@gmail.com>
@taiki-e taiki-e removed their assignment Oct 13, 2019
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.

1 participant