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

#195: calculate total number of CI builds #344

Merged
merged 1 commit into from
Sep 23, 2024
Merged

Conversation

Yegorov
Copy link
Contributor

@Yegorov Yegorov commented Sep 21, 2024

Closes #195

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 21, 2024

@Suban05 please review

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 fine

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 22, 2024

@yegor256 please check

@yegor256 yegor256 merged commit 62bb505 into zerocracy:master Sep 23, 2024
8 checks passed
@yegor256
Copy link
Member

@Yegorov thanks!

@0crat
Copy link

0crat commented Sep 23, 2024

@Suban05 Thank you for your code review contribution! We appreciate your effort, but there's room for improvement. According to our policy, you've earned +4 points as a base reward. However, we had to deduct 10 points due to the low number of comments (only 1). To ensure a minimum reward, we've adjusted the total to +4 points. Remember, more detailed feedback helps the team grow. Your current balance stands at +111. Keep up the good work and aim for more comprehensive reviews in the future! 💻👍

@0crat
Copy link

0crat commented Sep 23, 2024

@Yegorov Hey there! 👋 Great job on your contribution! You've earned +4 points for this one. Here's the breakdown: +4 base points, -4 because we only had 2 comments (we aim for at least 6), 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 - you're doing great! Your current balance is +149. 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_builds_ran into QoD
4 participants