Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Rename #[async_stream] to #[stream] #42

Closed
taiki-e opened this issue May 2, 2020 · 0 comments · Fixed by #45
Closed

Rename #[async_stream] to #[stream] #42

taiki-e opened this issue May 2, 2020 · 0 comments · Fixed by #45
Labels
breaking-change This proposes a breaking change C-enhancement Category: A new feature or an improvement for an existing one
Milestone
v0.2

Comments

@taiki-e
Copy link
Owner

taiki-e commented May 2, 2020

"async_" prefix may be redundant, since #[async_stream] can only be used with the async function.
@taiki-e taiki-e added the breaking-change This proposes a breaking change label May 2, 2020
@taiki-e taiki-e added this to the v0.2 milestone May 2, 2020
@taiki-e taiki-e mentioned this issue May 4, 2020
Merged
@bors bors bot closed this as completed in ad9196c May 4, 2020
@bors bors bot closed this as completed in #45 May 4, 2020
@taiki-e taiki-e added the C-enhancement Category: A new feature or an improvement for an existing one label Sep 21, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
breaking-change This proposes a breaking change C-enhancement Category: A new feature or an improvement for an existing one
Projects
None yet
Milestone
v0.2
Development

Successfully merging a pull request may close this issue.

Remove 'async_' prefix from macros taiki-e/futures-async-stream
1 participant
@taiki-e