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

#200: calculate total number of issues and pull requests #348

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

Yegorov
Copy link
Contributor

@Yegorov Yegorov commented Sep 23, 2024

Closes #200

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 23, 2024

@Suban05 please review. Test fail, because this zerocracy/fbe#101 PR need merge before

Copy link
Contributor

@Suban05 Suban05 left a comment

Choose a reason for hiding this comment

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

@Yegorov looks good

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 24, 2024

@yegor256 please check

@yegor256 yegor256 merged commit e66ce62 into zerocracy:master Sep 24, 2024
7 of 8 checks passed
@yegor256
Copy link
Member

@Yegorov thanks!

@0crat
Copy link

0crat commented Sep 24, 2024

@Suban05 Hey there! 👋 Great job on the review, but we've got some room for improvement. You snagged +4 points as a base, but lost 10 due to the low comment count (only 1). We bumped you up 10 points to keep things positive. Remember, more comments can earn you up to 8 extra points! Your current balance is +119. Keep up the good work and aim for those bonus points next time! 💪

@0crat
Copy link

0crat commented Sep 24, 2024

@Yegorov Hey there! 👋 Thanks for your contribution! You've earned +4 points this time. Here's the breakdown: +4 base points, -4 because there were only 3 comments (we like to see more discussion!), and +4 to ensure you get something for your effort. Remember, more hits-of-code and active code reviews can boost your points next time! Keep the contributions coming - we value your work! Your current balance is +170. Looking forward to your next submission! 💻🚀

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.

total_issues and total_pulls into DoT
4 participants