PR pickup time is considering any comment rather than review comment

[AnkeshThakur]

The PR pickup time calculation in DORA does not account for review comment. It picks any type of comment for calculation. Any automation (CI/bots etc) can comment on the PR and it would consider it to be picked up.

Code - https://github.com/apache/incubator-devlake/blob/e0584d67f543ec4e736950e51792922779e4d759/backend/plugins/dora/tasks/change_lead_time_calculator.go#L183-L191

It should filter based on comment type 'REVIEW' and first review comment's timestamp should be used for pickup time calculation. In case there is no review comment, pickup time should be equal to time diff between created_date and merge date.

[klesh]

Make sense, would you like to put up an PR for that?

[AnkeshThakur]

Yes. Sure thing!

[klesh]

Nice, looking forward to it.

[khalibartan]

is this fixed?