Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Modifying the "Reduce ITP with number of active tests per user" #1903

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

FauziAkram
Copy link
Contributor

@FauziAkram FauziAkram commented Feb 27, 2024

Closed it when fishtest was inactive, now re-opened it.

Reasons for this change:

Support for Active Developers:

Penalizing active developers with stringent bounds may discourage contributions. The proposed change ensures a more lenient approach, fostering a supportive environment for developers.

Balancing Act:

The adjustment strikes a balance between encouraging active testing and maintaining reasonable control over the number of tests. It provides a smoother reduction in itp as the number of active tests increases, offering a fair compromise.

Community Feedback:

I suggest seeking feedback from other devs on this proposed change. Open discussions can provide valuable insights and help in refining the formula to better suit the collective needs.

@mstembera
Copy link

mstembera commented Feb 27, 2024

Linking a bit of discussion on this same PR we already had. #1874 Not sure why we are starting a new PR when the original already started the discussion.

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