Skip to content

gerrit_project: allow slashes in repo path of gerrit remote#74

Merged
9999years merged 1 commit into9999years:mainfrom
Ma27:slashes
Feb 9, 2026
Merged

gerrit_project: allow slashes in repo path of gerrit remote#74
9999years merged 1 commit into9999years:mainfrom
Ma27:slashes

Conversation

@Ma27
Copy link
Copy Markdown
Contributor

@Ma27 Ma27 commented Feb 6, 2026

Otherwise, remote URLs such as ssh://ma27@cl.afnix.fr:2022/rokc/rokc are rejected by this tool.

cc @9999years

Otherwise, remote URLs such as `ssh://ma27@cl.afnix.fr:2022/rokc/rokc`
are rejected by this tool.
@github-actions github-actions bot added the patch label Feb 6, 2026
Copy link
Copy Markdown
Owner

@9999years 9999years left a comment

Choose a reason for hiding this comment

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

sweet tysm, honestly this regex is still super aggressive and maybe should just be like .+ or something? but it's been a while since i've looked at the code. now we get to find out if my release automation works!

@9999years 9999years merged commit 3002462 into 9999years:main Feb 9, 2026
28 checks passed
@9999years
Copy link
Copy Markdown
Owner

yeah the release automation is cooked. i'll fix it eventually

@Ma27 Ma27 deleted the slashes branch February 10, 2026 11:23
@stagnation
Copy link
Copy Markdown
Contributor

Nice, I have some similar small tweaks in a stack lying around as well that I hope to get around to submitting.
I really should have done so a long time ago :)

@9999years
Copy link
Copy Markdown
Owner

@Ma27 Got the release automation fixed, your change is published here! https://github.com/9999years/git-gr/releases/tag/v1.4.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants