You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi! Awesome tool, I could definitely see us using it.
One thing that makes it sort of hard to use (at least for our org at the moment, which is why I thought the metrics were wrong in that other issue :) ) is that we have CI users that respond to PRs. This messes with the time to response calculations as they are always pretty quick to respond, and thus beat the human response most of the time. I've tested by removing the first_comment_time from this line and re-running to get the actual time to first response (at least for us, assuming that a review comment is the first response is an ok assumption, I understand that isn't always true for everyone that would use this tool.)
My proposed solution would be to add a list of Users to ignore comments from, but I'm not really a Python person and can't tell how easy that would be to implement, since it looks like now you just pull the first comment from each issue, pr or discussion.
The text was updated successfully, but these errors were encountered:
Hi! Awesome tool, I could definitely see us using it.
One thing that makes it sort of hard to use (at least for our org at the moment, which is why I thought the metrics were wrong in that other issue :) ) is that we have CI users that respond to PRs. This messes with the time to response calculations as they are always pretty quick to respond, and thus beat the human response most of the time. I've tested by removing the
first_comment_time
from this line and re-running to get the actual time to first response (at least for us, assuming that a review comment is the first response is an ok assumption, I understand that isn't always true for everyone that would use this tool.)My proposed solution would be to add a list of Users to ignore comments from, but I'm not really a Python person and can't tell how easy that would be to implement, since it looks like now you just pull the first comment from each issue, pr or discussion.
The text was updated successfully, but these errors were encountered: