gitUrl support for Gerrit #42639
Replies: 3 comments 5 replies
-
|
Supporting I'm just not sure a new API request is worth it. We could construct the URL based on what user provides via Anyway, definitely send a PR. I do have other PRs that would likely conflict, but we can sort it out depending on whichever is merged first. |
Beta Was this translation helpful? Give feedback.
-
|
Thanks for the quick response @felipecrs! The reason I think we have to use an API request to get the clone URL is because (at least on our server) the port is non-standard. I'll send the PR now, happy to work through conflicts with you if/when that happens. |
Beta Was this translation helpful? Give feedback.
-
|
No
…On Wednesday, 15 April 2026, Felipe Santos ***@***.***> wrote:
The reason I think we have to use an API request to get the clone URL is
because (at least on our server) the port is non-standard.
Meaning it's not 29418?
—
Reply to this email directly, view it on GitHub
<#42639 (reply in thread)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/CBXT7IBMMPFWMWQ4T4PYJL34V76XFAVCNFSM6AAAAACXZXRPTOVHI2DSMVQWIX3LMV43URDJONRXK43TNFXW4Q3PNVWWK3TUHMYTMNJXGU4DINA>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***
com>
|
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Our local Gerrit server has disabled HTTPS clone operations. We have to use SSH to clone, and Gerrit has a helpful link to do this in the web UI in the General tab.
I would like the existing Gerrit platform support to read the server config to see what download schemes are supported, and use ssh if configured by the existing gitUrl parameter.
I have a fork that has implemented this, which seems to work well: https://github.com/spmiller/renovate/tree/feat-gerrit-gitUrl
I can see that @felipecrs has some Gerrit WIP, and I don't want to step on any toes. Is this change something that would be suitable for a PR?
Beta Was this translation helpful? Give feedback.
All reactions