Release/Revision Change_Requests_Duration.md

[klumb]

This issue was created to collect comments about the upcoming metrics release.

This thread is for comments about Change_Requests_Duration.md

GitHub location: https://github.com/chaoss/wg-evolution/blob/master/metrics/Change_Requests_Duration.md

See all release candidates of metrics are at: https://chaoss.community/metrics/

[klumb]

Name change revision - From "reviews to "change request"

[foundjem]

Change Request LGTM!

Signed-Off-By: Armstrong [email protected]

[rpaik]

Is there a reason why we would not consider PRs/MRs that are not accepted? If a change request is not accepted for valid reasons (e.g. the change would introduce a bug), I think it makes sense to include that change request if the goal of this metrics is to see how responsive reviewers are to change requests.

[foundjem]

@rpaik, are you suggesting that we include change requests that were not accepted? If that is the case, we may have to consider abandoned circumstances, which will make it almost impossible to compute the duration.

Signed-Off-By: Armstrong [email protected]

[rpaik]

@foundjem how about if we consider change requests that have either been accepted or closed? The abandoned change requests will then be excluded.

[klumb]

closing for release

May want to address this last comment in the next release

[ElizabethN]

We need to revisit this metric. The question does not seem to match the rest of the metric definition and when looking through the above comments on this issue, we should clarify the "abandoned requests" nuance as mentioned above.