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

Dynamic voting window 🚀 #309

Merged
merged 7 commits into from
May 27, 2017
Merged

Dynamic voting window 🚀 #309

merged 7 commits into from
May 27, 2017

Conversation

flibustier
Copy link
Contributor

This PR bounces on suggestions made in #275

That is to say :

  • The voting window is now the same as usual (this is now call as "initial_voting_window")
  • Two reasons can now extend the voting window (call as "extended_voting_window") :
    « There are two scenarios that the voting window is needed to be long; where it is being hotly debated, and where it gets no votes for many hours. »
    And what is the value of the "extending_voting_window" ?
    The value depends of the time since the creation date of the repo, following this rule :

save 2

When the repository is less than 3 days, the extending voting windows is 3 hours
When the repository is more stable and advanced, 16 days or more, it's 9 hours

@chaosbot
Copy link
Collaborator

🙆‍♀️ PR passed with a vote of 11 for and 0 against, with a weighted total of 11.0 and a threshold of 6.2.

See merge-commit 23ca855 for more details.

@chaosbot chaosbot merged commit 23ca855 into Chaosthebot:master May 27, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants