Skip to content

Add TimeStatsAsync to MergeRequestClient#543

Merged
Toa741 merged 4 commits into
ubisoft:mainfrom
Delphin-Technology:mergerequests-timestatsasync
Oct 25, 2023
Merged

Add TimeStatsAsync to MergeRequestClient#543
Toa741 merged 4 commits into
ubisoft:mainfrom
Delphin-Technology:mergerequests-timestatsasync

Conversation

@DelphinRP

Copy link
Copy Markdown
Contributor

Adds TimeStatsAsync to MergeRequestClient allowing fetching time stats for merge requests via

var timestats = await _gitLabClient.MergeRequests.TimeStatsAsync(mergeRequest.ProjectId, mergeRequest.Iid, CancellationToken.None);

Closes #538

@DelphinRP
DelphinRP requested a review from a team as a code owner October 13, 2023 08:05
@DelphinRP
DelphinRP requested review from Toa741 and removed request for a team October 13, 2023 08:05
Comment thread NGitLab/Impl/MergeRequestClient.cs Outdated
@Toa741
Toa741 enabled auto-merge (squash) October 20, 2023 19:49
@DelphinRP
DelphinRP force-pushed the mergerequests-timestatsasync branch from da861e4 to 602c44c Compare October 23, 2023 05:44
auto-merge was automatically disabled October 24, 2023 05:51

Head branch was pushed to by a user without write access

@DelphinRP
DelphinRP force-pushed the mergerequests-timestatsasync branch from 602c44c to 1e392fd Compare October 24, 2023 05:51
@DelphinRP

Copy link
Copy Markdown
Contributor Author

@Toa741 The branch getting outdated with commits to the main branch prevents auto merge from triggering.

@DelphinRP

Copy link
Copy Markdown
Contributor Author

@Toa741 @phdesroUbi @meziantou Is there anything we can do to speed up the merge of this pull request? Review is done, but the auto merge is not going through...

@Toa741
Toa741 merged commit 1ba8140 into ubisoft:main Oct 25, 2023
@Toa741

Toa741 commented Oct 25, 2023

Copy link
Copy Markdown
Contributor

@Toa741 @phdesroUbi @meziantou Is there anything we can do to speed up the merge of this pull request? Review is done, but the auto merge is not going through...

@DelphinRP, my bad, sorry for the delay

@DelphinRP
DelphinRP deleted the mergerequests-timestatsasync branch October 26, 2023 06:28
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.

Missing TimeStatsAsync for MergeRequests

2 participants