feat(cli): add clone command to download a standalone project#137
Merged
andrius-puksta-sensmetry merged 30 commits intomainfrom Dec 12, 2025
Merged
feat(cli): add clone command to download a standalone project#137andrius-puksta-sensmetry merged 30 commits intomainfrom
clone command to download a standalone project#137andrius-puksta-sensmetry merged 30 commits intomainfrom
Conversation
5bba64e to
2c5bd6d
Compare
c5d69f0 to
6303f0f
Compare
a1929a2 to
8672f58
Compare
…ok_or_else() instead of `ok_or()` to avoid unnecessary work in happy path Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> fixes, formatting Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> more file scheme tests Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> a Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…ommand Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…e instead of taking a String and then maybe converting it to Iri at some later point Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…fer to any dependencies, direct or indirect Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…ere possible Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
ccf0682 to
47ed7d8
Compare
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…ble if none is given, add error bound to all errors that matches anyhow's bound for convenience Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
It was unused, incurred an allocation and could easily be obtained by users of lock functions anyway. Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
c5938b3 to
2d88f28
Compare
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
2d88f28 to
bb614c5
Compare
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Collaborator
Author
|
PR is done and waiting for final review. |
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
…ets mistaken for locator) Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
vytautas-astrauskas-sensmetry
approved these changes
Dec 11, 2025
Collaborator
vytautas-astrauskas-sensmetry
left a comment
There was a problem hiding this comment.
Thanks!
Collaborator
|
A basic integration test shouldn't be too much hassle to add. |
…if dependency resolution fails Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
bad13eb to
ef079de
Compare
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Collaborator
Author
|
Added some integration tests for |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
clonecommand to download a standalone project. Closes Implement "editable" mode #129.sysand clone --helpoutput:Installing a project with std lib deps:
Cloning from path:
The project is directly cloned into the target directory. Any created files/dirs are cleaned up on failure.
clone_projectreturn project's info and metadata, as it's freeFixes
env installchoose latest valid version if none is specifiedOption::ok_or_else()instead ofok_or()for error reporting to avoid unnecessary work in happy pathDoc book updates will be done in a subsequent PR to reduce PR sizes.