Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

Implement orb sync subcommand #107

Merged
merged 3 commits into from
Oct 24, 2022
Merged

Implement orb sync subcommand #107

merged 3 commits into from
Oct 24, 2022

Conversation

cdata
Copy link
Collaborator

@cdata cdata commented Oct 24, 2022

This change implements the orb sync subcommand, which represents the first plausibly end-user-accessible path to accessing the Noosphere across multiple of their own devices.

I recorded an end-to-end demo of multi-device synchronization in action last week:

orb sync demo

Note that the demo was recorded against work in progress, and should not be considered a full-fidelity representation of the change in this branch.

In addition to the enhancements in this change, I also ran cargo clippy --fix as a one-off to clean up our outstanding warnings and lint infractions.

Follow-on work is captured as issues in ucan-wg/rs-ucan#37, ucan-wg/rs-ucan#32, #102, #103, #104, #105 and #106.

Fixes #87
Fixes #75

@cdata cdata requested a review from jsantell October 24, 2022 19:12
jsantell
jsantell previously approved these changes Oct 24, 2022
Copy link
Contributor

@jsantell jsantell left a comment

Choose a reason for hiding this comment

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

Looks great ✨ just a few comments from a high level

rust/noosphere-api/src/data.rs Show resolved Hide resolved
rust/noosphere-cli/src/native/commands/sphere.rs Outdated Show resolved Hide resolved
rust/noosphere-cli/src/native/workspace.rs Outdated Show resolved Hide resolved
@cdata cdata merged commit 81e9618 into main Oct 24, 2022
@cdata cdata deleted the feature/orb-sync branch October 24, 2022 23:14
@cdata cdata mentioned this pull request Oct 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

noosphere-cli should note and preserve original file extension in a header Implement orb sync subcommand
2 participants