search

cncf / devstats

📈CNCF-created tool for analyzing and graphing developer contributions
https://devstats.cncf.io
Apache License 2.0
61 stars 22 forks source link

[feature request] add PR reviews category to dashboards #35

Closed deepthi closed 9 months ago

deepthi commented 9 months ago

Currently devstats solely tracks PR authors and show PR author contributions by Company etc. A big part of project maintainer work is PR reviews. In fact, security guidelines require two reviews per pull request before it can be accepted. These contributions are currently being ignored by devstats. Code review is also development work, and should be captured.

lukaszgryglicki commented 9 months ago

Reviews are being counted as contributions, where do you think they are ignored? Can you point me to some examples - if they are ignored, then this is a bug and I will fix it.

lukaszgryglicki commented 9 months ago

@deepthi ping - where / which dashboard - reviews are ignored?

lukaszgryglicki commented 9 months ago

Closing this because a got no additional feedback, please reopen if needed.

deepthi commented 9 months ago

You are right, I explored the dashboards and found where they are counted. Thanks! Great work.