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

deps: promote tokio to workspace dependency #5251

Merged
merged 5 commits into from
Apr 15, 2024

Conversation

drHuangMHT
Copy link
Contributor

@drHuangMHT drHuangMHT commented Mar 21, 2024

Description

Promote tokio to workspace dependency.
Related issue: #5230

Notes & open questions

Not sure if setting default-features to false on workspace level causes any problem.

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

@drHuangMHT drHuangMHT marked this pull request as ready for review April 7, 2024 05:52
@drHuangMHT
Copy link
Contributor Author

@jxs Ready for review.

@drHuangMHT
Copy link
Contributor Author

Ping @jxs got a minute to review this?

transports/dns/src/lib.rs Show resolved Hide resolved
@jxs
Copy link
Member

jxs commented Apr 13, 2024

Thanks for this! LGTM only one question

@jxs jxs added the send-it label Apr 15, 2024
@mergify mergify bot merged commit bee8199 into libp2p:master Apr 15, 2024
71 of 72 checks passed
@drHuangMHT drHuangMHT deleted the tokio-workspace branch April 22, 2024 00:26
umgefahren pushed a commit to p2p-industries/rust-libp2p that referenced this pull request May 20, 2024
Promote `tokio` to workspace dependency.

Pull-Request: libp2p#5251.
umgefahren pushed a commit to p2p-industries/rust-libp2p that referenced this pull request May 20, 2024
Promote `tokio` to workspace dependency.

Pull-Request: libp2p#5251.
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.

None yet

2 participants