-
Notifications
You must be signed in to change notification settings - Fork 5
Fixed most of the issues and also adds much more fun! #8
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
base: master
Are you sure you want to change the base?
Conversation
|
This is a pretty cool update. A couple thoughts, though:
|
|
|
👍 for pushups, i want that unbalanced no-leg-day thing everyone seems to be doin! |
|
@robinpowered/product-team Merge PLS, this adds double pushups for us |
|
Not to speak for too many, but I know that most of the people that do the pushups together in the office simply do a 1-to-1 on the number of open pull requests (25 pushups for 25 PRs, regardless of PR age). Is there any way we can make the non-linear pushup calculation (and old count) an option? |
So this PR implements #7, #4, #5, #1, #2.
Fixes pluralization issues, and add fun! Now messages might looks like this:
