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

batches: follow up from #44458#45003

Merged
adeola-ak merged 4 commits into
mainfrom
aa/gitlab-update
Dec 5, 2022
Merged

batches: follow up from #44458#45003
adeola-ak merged 4 commits into
mainfrom
aa/gitlab-update

Conversation

@adeola-ak

Copy link
Copy Markdown
Contributor

This is a follow up to Adam's feedback from #44458

Test plan

none required

@cla-bot cla-bot Bot added the cla-signed label Dec 1, 2022
@adeola-ak adeola-ak changed the title batches: address missed PR feedback batches: follow up from #44458 Dec 1, 2022
@sourcegraph-bot

sourcegraph-bot commented Dec 1, 2022

Copy link
Copy Markdown
Contributor

Codenotify: Notifying subscribers in CODENOTIFY files for diff 02d3b43...bb82e79.

Notify File(s)
@eseliger enterprise/internal/batches/sources/gitlab.go
internal/extsvc/gitlab/mock.go
internal/extsvc/gitlab/projects.go
internal/extsvc/gitlab/projects_test.go

@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.

Thanks for taking this on!

Comment thread internal/extsvc/gitlab/projects.go Outdated
@adeola-ak adeola-ak merged commit e747e78 into main Dec 5, 2022
@adeola-ak adeola-ak deleted the aa/gitlab-update branch December 5, 2022 16:30
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.

4 participants