bujjibabukatta commented on code in PR #8928:
URL: https://github.com/apache/devlake/pull/8928#discussion_r3466831602
##########
backend/plugins/gitlab/tasks/mr_detail_collector.go:
##########
@@ -62,6 +62,7 @@ func CollectApiMergeRequestDetails(taskCtx
plugin.SubTaskContext) errors.Error {
Query: func(reqData *helper.RequestData) (url.Values,
errors.Error) {
query := url.Values{}
query.Set("with_stats", "true")
+ query.Set("include_diff_stats", "true")
Review Comment:
Hey @klesh, good catch!
Looked it up — `diff_stats` has been around since GitLab 13.0 (June 2020),
so most instances should be fine. For anything older, I've made `DiffStats` a
pointer so we can tell when it's missing and fall back to `changes_count`
instead of storing zeros. And `include_diff_stats=true` in the query is
harmless on old versions — they just ignore params they don't know.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
