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(iroh-base): introduce an Arc into RelayUrl #3065

Merged
merged 3 commits into from
Dec 19, 2024

Conversation

dignifiedquire
Copy link
Contributor

@dignifiedquire dignifiedquire commented Dec 19, 2024

Description

This makes the cloneing of RelayUrl much cheaper.
It turned out the whole code base was already using the RelayUrl as an immutable type already, so no changes otherwise necessary.

Breaking Changes

Notes & open questions

Change checklist

  • Self-review.
  • Documentation updates following the style guide, if relevant.
  • Tests if relevant.
  • All breaking changes documented.

Copy link

github-actions bot commented Dec 19, 2024

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/3065/docs/iroh/

Last updated: 2024-12-19T12:47:26Z

Copy link

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: 1046662

Copy link
Contributor

@flub flub left a comment

Choose a reason for hiding this comment

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

🎉

@dignifiedquire dignifiedquire added this pull request to the merge queue Dec 19, 2024
Merged via the queue into main with commit 834ab78 Dec 19, 2024
26 checks passed
@dignifiedquire dignifiedquire deleted the feat-arc-relay-url branch December 19, 2024 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants