Skip to content
This repository has been archived by the owner on Jun 21, 2020. It is now read-only.

Update futures-preview to 0.3.0-alpha.16 #28

Merged
merged 2 commits into from
May 11, 2019
Merged

Conversation

taiki-e
Copy link
Contributor

@taiki-e taiki-e commented May 9, 2019

Description

await syntax was implemented in rust-lang/rust#60586, and nightly-2019-05-09 has been released with the changes. Also, await! macro will be removed in the future.

Then, futures 0.3.0-alpha.16 was released.

Refs: migration tool

Motivation and Context

How Has This Been Tested?

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

With this change, the minimum required version will go up to nightly-2019-05-09.

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@taiki-e taiki-e changed the title Replace await! macro with await syntax Update futures-preview to 0.3.0-alpha.16 May 10, 2019
@taiki-e
Copy link
Contributor Author

taiki-e commented May 10, 2019

futures 0.3.0-alpha.16 was released, so I added a commit and updated this PR.

Copy link
Member

@yoshuawuyts yoshuawuyts left a comment

Choose a reason for hiding this comment

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

Fantastic, thanks so much!! Will release after the weekend!

@yoshuawuyts yoshuawuyts merged commit 1174639 into http-rs:master May 11, 2019
@taiki-e taiki-e deleted the await branch May 11, 2019 17:53
@taiki-e taiki-e restored the await branch May 11, 2019 17:55
@taiki-e taiki-e deleted the await branch May 12, 2019 01:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants