wouldd commented on issue #7193:
URL:
https://github.com/apache/incubator-devlake/issues/7193#issuecomment-2066843305
right I guess I meant that where you said 'The con is that the very first
deployment will not be linked to any PRs.' I thought that might be avoidable -
in my case the deployment webhook is still being told which commits are
associated and those commits can be associated to the pr they went through etc.
It's still not entirely clear to me what I am losing/missing by just
updating that dal filter in the change_leadtime_calculator in the
getDeploymentscommit method to be:
dal.Where("pm.project_name = ? AND **cd.new_commit_sha** = ? AND dc.RESULT =
?", projectName, mergeSha, devops.RESULT_SUCCESS),
instead of cd.commit_sha.
it certainly appears to me that the stats start to make sense when I run
that way
--
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]
