Skip to content

fix: harden GitLab merge request handling - #341

Merged
eshulman2 merged 2 commits into
mainfrom
feat/restore-gitlab-source-control
Sep 9, 2026
Merged

fix: harden GitLab merge request handling#341
eshulman2 merged 2 commits into
mainfrom
feat/restore-gitlab-source-control

Conversation

@eshulman2

@eshulman2 eshulman2 commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Restore the GitLab source-control integration from Add GitLab adapter #314 on the main-based branch.
  • Normalize configured self-managed GitLab API URLs and document accepted forms.
  • Preserve REST merge-request head SHAs and source-project cache keys.
  • Treat allowed GitLab job failures as neutral and make 409 merge-request recovery fork-safe.

Validation

  • 195 targeted GitLab and source-control tests passed.
  • Ruff passed for all changed Python files.
  • git diff --check passed.

Note

The full suite was attempted but timed out after 55 seconds at the previously isolated, environment-specific synchronous FastAPI TestClient hang in the GitLab webhook route tests; no assertion failure was reported.

@eshulman2
eshulman2 merged commit 37deabb into main Sep 9, 2026
8 checks passed
@eshulman2
eshulman2 deleted the feat/restore-gitlab-source-control branch September 9, 2026 06:47
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.

1 participant