Skip to content

feat(swarm): remove KeepAlive::Until from OneshotHandler#4677

Merged
mergify[bot] merged 7 commits intolibp2p:masterfrom
leonzchang:feat/one-shot-remove-keep-until
Oct 20, 2023
Merged

feat(swarm): remove KeepAlive::Until from OneshotHandler#4677
mergify[bot] merged 7 commits intolibp2p:masterfrom
leonzchang:feat/one-shot-remove-keep-until

Conversation

@leonzchang
Copy link
Copy Markdown
Contributor

Description

Related: #3844.
Related: #4656.

Notes & open questions

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • A changelog entry has been made in the appropriate crates

@leonzchang leonzchang marked this pull request as ready for review October 17, 2023 08:55
Copy link
Copy Markdown
Contributor

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

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

Awesome, thank you!

Just one comment.

@thomaseizinger thomaseizinger marked this pull request as draft October 17, 2023 09:22
@thomaseizinger thomaseizinger changed the title feat(one-shot): remove KeepAlive::Until feat(swarm): remove KeepAlive::Until from OneshotHandler` Oct 17, 2023
@thomaseizinger thomaseizinger changed the title feat(swarm): remove KeepAlive::Until from OneshotHandler` feat(swarm): remove KeepAlive::Until from OneshotHandler Oct 17, 2023
mergify bot pushed a commit that referenced this pull request Oct 17, 2023
@thomaseizinger thomaseizinger added this to the v0.53.0 milestone Oct 17, 2023
@mergify

This comment was marked as resolved.

@thomaseizinger thomaseizinger marked this pull request as ready for review October 20, 2023 01:18
@mergify
Copy link
Copy Markdown
Contributor

mergify bot commented Oct 20, 2023

This pull request has merge conflicts. Could you please resolve them @leonzchang? 🙏

@mergify mergify bot merged commit b1b46d1 into libp2p:master Oct 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants