Skip to content

Clarify adding SSH Git dependencies #13534

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 2 commits into from
May 20, 2025
Merged

Conversation

art-dsit
Copy link
Contributor

The current instructions say

prefix a Git-compatible URL (i.e., that you would use with git clone) with git+.

But this does not work with the URL that Github gives you when you choose Clone -> SSH via the UI, which is of the form git@github.com:astral-sh/uv.git. If you prefix this with git+, i.e.

git+git@github.com:astral-sh/uv.git

it does not work.

art-dsit and others added 2 commits May 19, 2025 14:17
The current instructions say 

> prefix a Git-compatible URL (i.e., that you would use with git clone) with git+.

But this does not work with the URL that Github gives you when you choose Clone -> SSH via the UI, which is of the form `git@github.com:astral-sh/uv.git`. If you prefix this with `git+`, i.e.

`git+git@github.com:astral-sh/uv.git`

it does not work.
Copy link
Member

@charliermarsh charliermarsh left a comment

Choose a reason for hiding this comment

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

Thanks!

@charliermarsh charliermarsh added the documentation Improvements or additions to documentation label May 20, 2025
@charliermarsh charliermarsh enabled auto-merge (squash) May 20, 2025 14:15
@charliermarsh charliermarsh merged commit 1bee932 into astral-sh:main May 20, 2025
86 checks passed
tmeijn pushed a commit to tmeijn/dotfiles that referenced this pull request May 23, 2025
This MR contains the following updates:

| Package | Update | Change |
|---|---|---|
| [astral-sh/uv](https://github.com/astral-sh/uv) | patch | `0.7.6` -> `0.7.7` |

MR created with the help of [el-capitano/tools/renovate-bot](https://gitlab.com/el-capitano/tools/renovate-bot).

**Proposed changes to behavior should be submitted there as MRs.**

---

### Release Notes

<details>
<summary>astral-sh/uv (astral-sh/uv)</summary>

### [`v0.7.7`](https://github.com/astral-sh/uv/blob/HEAD/CHANGELOG.md#077)

[Compare Source](astral-sh/uv@0.7.6...0.7.7)

##### Python

-   Work around third-party packages that (incorrectly) assume the interpreter is dynamically linking libpython
-   Allow the experimental JIT to be enabled at runtime on Python 3.13 and 3.14 on macOS on aarch64 aka Apple Silicon

See the
[`python-build-standalone` release notes](https://github.com/astral-sh/python-build-standalone/releases/tag/20250521)
for more details.

##### Bug fixes

-   Make `uv version` lock and sync ([#&#8203;13317](astral-sh/uv#13317))
-   Fix references to `ldd` in diagnostics to correctly refer to `ld.so` ([#&#8203;13552](astral-sh/uv#13552))

##### Documentation

-   Clarify adding SSH Git dependencies ([#&#8203;13534](astral-sh/uv#13534))

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever MR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this MR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this MR, check this box

---

This MR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiI0MC4yMy4xIiwidXBkYXRlZEluVmVyIjoiNDAuMjMuMSIsInRhcmdldEJyYW5jaCI6Im1haW4iLCJsYWJlbHMiOlsiUmVub3ZhdGUgQm90Il19-->
@brian316
Copy link

ive come across many instances where users dont know to remove the colon (:) and replace with a forward slash (/) for ssh urls, would that be useful hint in the docs?

e.g.

FROM:
git+git@github.com:astral-sh/uv.git

TO:
git+git@github.com/astral-sh/uv.git

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants