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

refactor: inline quicksink crate #4711

Merged
merged 4 commits into from
Oct 24, 2023
Merged

refactor: inline quicksink crate #4711

merged 4 commits into from
Oct 24, 2023

Conversation

kayabaNerve
Copy link
Contributor

Description

Adds the quicksink crate, which is offered under the MIT or Apache 2 license, as a module. This enables maintenance since quicksink itself has been archived for a year in a half.

Resolves #4705.

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

@kayabaNerve kayabaNerve changed the title Add quicksink as a mod chore: Add quicksink as a mod Oct 23, 2023
@mergify
Copy link
Contributor

mergify bot commented Oct 23, 2023

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

Resolves #4705 without an increase in semver burden.

Decrease Cargo.lock by one package and enables any/all future maintenance.
@kayabaNerve
Copy link
Contributor Author

Feel free to say this also shouldn't have a changelog entry.

@thomaseizinger thomaseizinger changed the title chore: Add quicksink as a mod refactor: inline quicksink crate Oct 23, 2023
transports/websocket/src/quicksink.rs Outdated Show resolved Hide resolved
transports/websocket/CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
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.

Nice, thank you!

@mergify mergify bot merged commit 2711106 into libp2p:master Oct 24, 2023
71 checks passed
@kayabaNerve kayabaNerve deleted the adopt-quicksink branch October 24, 2023 04:19
Copy link
Member

@mxinden mxinden left a comment

Choose a reason for hiding this comment

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

👍 for inlining as a module.

jxs pushed a commit to jxs/rust-libp2p that referenced this pull request Jul 11, 2024
Adds the quicksink crate, which is offered under the MIT or Apache 2 license, as a module. This enables maintenance since quicksink itself has been archived for a year in a half.

Resolves libp2p#4705.

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

Adopt quicksink
3 participants