Skip to content

Conversation

@Arunodoy18
Copy link

  • Standardize contribution thresholds across all developer statistics queries
  • Shared project stats: Reduce 'All' repository group thresholds from 10/15/20 to 5/5/10 contributions to match specific repository groups
  • Kubernetes project stats: Reduce 'All' repository group thresholds from 200/30 to 100/5 contributions for consistency
  • Ensures OpenTelemetry and other CNCF projects have consistent voter eligibility across different dashboard views
  • Fixes issue where 'All' repository group showed different developer counts than specific repository groups
  • Critical for election integrity: prevents eligible developers from being excluded due to inconsistent filtering
  • Aligns all query types to use lower, more inclusive thresholds while maintaining data quality

Please make sure that you follow instructions from CONTRIBUTING

Specially:

  • Check if all tests pass, see TESTING for deatils.
  • Make sure you've added test coverage for new features/metrics.
  • Make sure you have updated documentation.
  • If you added a new metric, please make sure you have been following instructions about adding new metric.

- Standardize contribution thresholds across all developer statistics queries
- Shared project stats: Reduce 'All' repository group thresholds from 10/15/20 to 5/5/10 contributions to match specific repository groups
- Kubernetes project stats: Reduce 'All' repository group thresholds from 200/30 to 100/5 contributions for consistency
- Ensures OpenTelemetry and other CNCF projects have consistent voter eligibility across different dashboard views
- Fixes issue where 'All' repository group showed different developer counts than specific repository groups
- Critical for election integrity: prevents eligible developers from being excluded due to inconsistent filtering
- Aligns all query types to use lower, more inclusive thresholds while maintaining data quality

Signed-off-by: Arunodoy18 <[email protected]>
@Arunodoy18
Copy link
Author

HI, i have completed the issue resolving regarding the issue , If anyone reviews it according to their time, could be great . Can inform me of any more corrections needed.
Thank you.

Copy link
Member

@lukaszgryglicki lukaszgryglicki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm but let's wait with merging it.
I will merge after infra update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants