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

#194: calculate total number of code reviews #339

Merged
merged 1 commit into from
Sep 17, 2024
Merged

Conversation

Yegorov
Copy link
Contributor

@Yegorov Yegorov commented Sep 16, 2024

Closes #194

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 16, 2024

@Suban05 please review. Test fail, because this zerocracy/fbe#97 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 17, 2024

@yegor256 please check

@yegor256 yegor256 merged commit 3158d66 into zerocracy:master Sep 17, 2024
8 checks passed
@yegor256
Copy link
Member

@Yegorov thanks!

@0crat
Copy link

0crat commented Sep 17, 2024

@Suban05 Great job on the review! 🌟 You've earned +4 points: +4 base, -10 for minimal comments, but +10 to recognize your effort. Your balance now stands at +250. Keep up the good work and aim for more detailed feedback next time!

@0crat
Copy link

0crat commented Sep 17, 2024

@Yegorov Hey there! 👋 Awesome job on your contribution! You've snagged +10 points for this one: 4 for showing up, and a sweet 9 for those 193 hits-of-code you cranked out. Just a heads up, we docked 4 points 'cause there were only 2 comments - we love seeing more chatter during reviews! Keep the code flowing and maybe aim for 6+ comments next time to avoid that deduction. Your balance is looking good at +180. Remember, quality and speed are our jam - let's keep this momentum going! 🚀

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_reviews_submitted into QoD
4 participants