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

feat(swarm): remove unused types from public API #3497

Merged
merged 3 commits into from
Feb 23, 2023

Conversation

thomaseizinger
Copy link
Contributor

Description

Notes

Links to any relevant issues

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

@mergify
Copy link
Contributor

mergify bot commented Feb 23, 2023

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

@thomaseizinger thomaseizinger changed the base branch from 2824-deprecate-into-connection-handler to master February 23, 2023 00:11
swarm/CHANGELOG.md Outdated Show resolved Hide resolved
swarm/CHANGELOG.md Outdated Show resolved Hide resolved
@thomaseizinger
Copy link
Contributor Author

@mxinden I assume your approval also means that you are okay with editing the changelog. I figured that removing intermediate entries is good to not confuse users unecessarily.

@mergify mergify bot merged commit 794b2a2 into master Feb 23, 2023
@mergify mergify bot deleted the 2824-remove-pending-error-api branch February 23, 2023 22:25
@mxinden
Copy link
Member

mxinden commented Feb 24, 2023

I am in favor of the changelog edits introduced here 👍 thanks for asking.

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