Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

batches: fix fork naming for gitlab#44458

Merged
adeola-ak merged 7 commits into
mainfrom
aa/fix-gitlab-forks
Nov 28, 2022
Merged

batches: fix fork naming for gitlab#44458
adeola-ak merged 7 commits into
mainfrom
aa/fix-gitlab-forks

Conversation

@adeola-ak

@adeola-ak adeola-ak commented Nov 16, 2022

Copy link
Copy Markdown
Contributor

Closes: #44456

This PR is to match the behavior and naming convention first implemented for bitbucket server (#43681) which will add the original namespace to the repo name when creating the fork on the codehost (gitlab) and fork to usernamespace/originalnamespace-reponame as well as update the path

Test plan

manually tested the creation of forks with enforce forks setting on

@adeola-ak adeola-ak marked this pull request as ready for review November 23, 2022 00:46
@adeola-ak adeola-ak requested a review from a team November 23, 2022 00:46
@sourcegraph-bot

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in CODENOTIFY files for diff f34562d...ad3a876.

Notify File(s)
@eseliger enterprise/internal/batches/sources/gitlab.go
internal/extsvc/gitlab/projects.go
internal/extsvc/gitlab/projects_test.go
internal/extsvc/gitlab/testdata/vcr/TestClient_ForkProject.yaml
internal/extsvc/gitlab/testdata/vcr/TestClient_ForkProject/already_forked.yaml
internal/extsvc/gitlab/testdata/vcr/TestClient_ForkProject/success.yaml

@LawnGnome LawnGnome left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One question about the exact signature of ForkProject, but otherwise LGTM.

Comment thread internal/extsvc/gitlab/projects.go
Comment thread internal/extsvc/gitlab/projects.go
@adeola-ak adeola-ak merged commit e42e2af into main Nov 28, 2022
@adeola-ak adeola-ak deleted the aa/fix-gitlab-forks branch November 28, 2022 02:36
adeola-ak added a commit that referenced this pull request Dec 5, 2022
clean up gitlab forks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

batches: unify the naming convention of forks created from batch changes for gitlab

3 participants