Skip to content

feat: cli #14

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

Merged
merged 6 commits into from
Nov 18, 2024
Merged

feat: cli #14

merged 6 commits into from
Nov 18, 2024

Conversation

rklaehn
Copy link
Collaborator

@rklaehn rklaehn commented Nov 18, 2024

Description

Add blobs cli from main iroh repo

Breaking Changes

None, this just adds stuff.

Notes & open questions

Currently we just require the caller to figure out the NodeAddr before invoking run. We could have some lazy way to figure out the NodeAddr, like a callback or a trait or even a generic node rpc interface as tried out in n0-computer/iroh#2927. But I think this is nice and simple for now.

Change checklist

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

caller needs to figure out the addr before invoking run
Copy link

github-actions bot commented Nov 18, 2024

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

Last updated: 2024-11-18T11:43:59Z

@rklaehn rklaehn marked this pull request as ready for review November 18, 2024 11:42
@rklaehn rklaehn merged commit cdcb863 into main Nov 18, 2024
24 checks passed
@rklaehn rklaehn deleted the cli branch November 18, 2024 13:23
github-merge-queue bot pushed a commit to n0-computer/iroh that referenced this pull request Nov 18, 2024
)

## Description

<!-- A summary of what this pull request achieves and a rough list of
changes. -->

## Breaking Changes

<!-- Optional, if there are any breaking changes document them,
including how to migrate older code. -->

## Notes & open questions

Node: depends on n0-computer/iroh-blobs#14

<!-- Any notes, remarks or open questions you have to make about the PR.
-->

## Change checklist

- [ ] Self-review.
- [ ] Documentation updates following the [style
guide](https://rust-lang.github.io/rfcs/1574-more-api-documentation-conventions.html#appendix-a-full-conventions-text),
if relevant.
- [ ] Tests if relevant.
- [ ] All breaking changes documented.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants