This repository was archived by the owner on Feb 25, 2026. It is now read-only.
Conversation
We want to support the more familiar https://foo.com/bar@master syntax to specify git references, but it requires special handling to avoid overlapping with url usernames. To solve this, parse the url with gix::url first, which can distinguish a username from a `@` in a url path, then parse to a UriRef so we can also expand aliases. Also fix distinguishing atom uris from urls more directly by simply searching for an atom's `::` delimiter.
Also includes wrapping cli and markdown support in help messages for better formatting.
still need sync code
The `@` delimiter is categorically ambiguous, so instead we will use a `^` since it is invalid in URLs.
In the case where the alias itself contained a path, and after the alias was an additional path, the remaining path was cut off and not detected leading to the incorrect url being parsed. Correct this.
Still missing manifest/lock synchronization, which would require some lightweight remote checks. The bulk of the dependency fetching is offloaded to snix, and the blobservice doubles as a lightweight fetcher cache via a simple http middleware service: nrdxp/snix@da5e4db
Fetches protos and sets up necessary vars for snix crates to "just build" inside of the devshell.
Owner
Author
|
Before we end up with another massive PR, I am gonna go ahead and merge, and will flesh out the remaining details in a follow up PR |
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
1 participant
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.
Implements the fetching and locking of pin-style dependencies. In general, this is one of the last concerns to finish off before we have a usable demo of eka's core features.