Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add TimeStatsAsync to MergeRequestClient #543

Merged
merged 4 commits into from
Oct 25, 2023

Conversation

DelphinRP
Copy link
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
@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
Contributor Author

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

@DelphinRP
Copy link
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
4 checks passed
@Toa741
Copy link
Contributor

Toa741 commented Oct 25, 2023

@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