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

Allow #[project] to be used on if let expressions #181

Merged
merged 6 commits into from
Mar 20, 2020

Conversation

Aaron1011
Copy link
Collaborator

This PR would work correctly, except for the fact that
rust-lang/rust#68618 causes a compilation error to be emitted before we
even have a chance to run. That issue is independent of the
implementation of this PR, so this PR should start working automatically
once the issue is resolved.

@Aaron1011 Aaron1011 requested a review from taiki-e as a code owner January 28, 2020 21:52
@taiki-e taiki-e added C-enhancement Category: A new feature or an improvement for an existing one S-blocked Status: Blocked on something else labels Jan 29, 2020
This PR would work correctly, except for the fact that
rust-lang/rust#68618 causes a compilation error to be emitted before we
even have a chance to run. That issue is independent of the
implementation of this PR, so this PR should start working automatically
once the issue is resolved.
@Aaron1011 Aaron1011 force-pushed the feature/project-if-let branch from 8920b02 to f626aa8 Compare March 9, 2020 15:31
@Aaron1011 Aaron1011 changed the title [WIP] Allow #[project] to be used on if let expressions Allow #[project] to be used on if let expressions Mar 9, 2020
@Aaron1011
Copy link
Collaborator Author

@taiki-e: This is ready for review. It will start compiling tomorrow, when rust-lang/rust#69201 becomes part of the next nightly.

The output of cargo fmt now has an error message (see rust-lang/rustfmt#4077), though it still seems to format the crate correctly. The error seems unrelated to this PR, and was likely caused by an update to rustfmt.

@Aaron1011 Aaron1011 force-pushed the feature/project-if-let branch from 8998a80 to 9910909 Compare March 10, 2020 15:51
@Aaron1011
Copy link
Collaborator Author

All tests are now passing with the latest nightly.

@Aaron1011
Copy link
Collaborator Author

@taiki-e: Are there any changes that you'd like me to make?

@taiki-e taiki-e removed the S-blocked Status: Blocked on something else label Mar 20, 2020
@taiki-e
Copy link
Owner

taiki-e commented Mar 20, 2020

Looks great, thank you!

bors r+

@bors
Copy link
Contributor

bors bot commented Mar 20, 2020

@bors bors bot merged commit 7081486 into taiki-e:master Mar 20, 2020
@Aaron1011 Aaron1011 deleted the feature/project-if-let branch March 22, 2020 16:44
bors bot added a commit to taiki-e/auto_enums that referenced this pull request Apr 14, 2020
89: Add tests for `#[attr] if` r=taiki-e a=taiki-e

Closes #86 

Refs: taiki-e/pin-project#181

Co-authored-by: Taiki Endo <[email protected]>
@taiki-e taiki-e added the A-project-attribute Area: #[project], #[project_ref], and #[project_replace] (note: this was removed in v1.0) label May 7, 2020
@taiki-e taiki-e removed the C-enhancement Category: A new feature or an improvement for an existing one label Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-project-attribute Area: #[project], #[project_ref], and #[project_replace] (note: this was removed in v1.0)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants