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

Remove lifetime from pinned self in trait #6

Merged
merged 1 commit into from
Sep 20, 2023
Merged

Remove lifetime from pinned self in trait #6

merged 1 commit into from
Sep 20, 2023

Conversation

notgull
Copy link
Member

@notgull notgull commented Sep 20, 2023

I forgot why I did this originally

I forgot why I did this originally

Signed-off-by: John Nunley <dev@notgull.net>
notgull added a commit to smol-rs/async-channel that referenced this pull request Sep 20, 2023
Make sure that smol-rs/event-listener-strategy#6 doesn't break the
other main use case.

Signed-off-by: John Nunley <dev@notgull.net>
@fogti fogti merged commit 27a14c4 into main Sep 20, 2023
9 checks passed
@taiki-e taiki-e deleted the notgull/no-lt branch September 20, 2023 13:26
@notgull notgull mentioned this pull request Sep 20, 2023
notgull added a commit to smol-rs/async-channel that referenced this pull request Sep 22, 2023
Make sure that smol-rs/event-listener-strategy#6 doesn't break the
other main use case.

Signed-off-by: John Nunley <dev@notgull.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants