Skip to content

add support for gitlab - #175

Merged
narnaud merged 2 commits into
narnaud:mainfrom
dfaure-kdab:wip/dfaure/gitlab
Jul 14, 2026
Merged

add support for gitlab#175
narnaud merged 2 commits into
narnaud:mainfrom
dfaure-kdab:wip/dfaure/gitlab

Conversation

@dfaure-kdab

Copy link
Copy Markdown
Collaborator

fix(trace): keep stderr for successful commands

Servers print MR/review links to stderr on a successful push, but the
trace only recorded stderr for failed commands, so those links were
dropped from both the summary and the log file.


feat(push): support GitLab and surface server MR/PR links

Plain pushes now display any remote: URL lines from the push output,
so GitLab's merge-request creation link is shown even when the remote
type is not detected (e.g. self-hosted invent.kde.org).

Add a GitLab remote type (config gitlab or a gitlab URL) that pushes
with merge_request.create push options so the server creates or links
the MR directly.

Servers print MR/review links to stderr on a successful push, but the
trace only recorded stderr for failed commands, so those links were
dropped from both the summary and the log file.
Plain pushes now display any remote: URL lines from the push output,
so GitLab's merge-request creation link is shown even when the remote
type is not detected (e.g. self-hosted invent.kde.org).

Add a GitLab remote type (config gitlab or a gitlab URL) that pushes
with merge_request.create push options so the server creates or links
the MR directly.
@dfaure-kdab
dfaure-kdab requested a review from narnaud as a code owner July 8, 2026 11:13

@narnaud narnaud left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Super nice.
The last one missing is gitea/forgejo, but it should be close to gitlab IMO.

@narnaud
narnaud merged commit 38acd1e into narnaud:main Jul 14, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants