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

judges-action#238: add search_commits in Fbe::FakeOctokit #108

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

Yegorov
Copy link
Contributor

@Yegorov Yegorov commented Sep 29, 2024

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 29, 2024

@Suban05 please review

@Yegorov
Copy link
Contributor Author

Yegorov commented Sep 30, 2024

@yegor256 please merge this pr

@yegor256 yegor256 merged commit b83ad50 into zerocracy:master Sep 30, 2024
9 checks passed
@yegor256
Copy link
Member

@rultor release, tag is 0.0.74

@rultor
Copy link
Contributor

rultor commented Sep 30, 2024

@rultor release, tag is 0.0.74

@yegor256 OK, I will release it now. Please check the progress here

@rultor
Copy link
Contributor

rultor commented Sep 30, 2024

@rultor release, tag is 0.0.74

@yegor256 Done! FYI, the full log is here (took me 8min)

@0crat
Copy link

0crat commented Sep 30, 2024

@Suban05 Great job on the review! 🌟 You've earned +4 points: +4 base, -10 for minimal comments, but +10 to ensure recognition. Your dedication is appreciated! Current balance: +130. Keep up the excellent work!

@0crat
Copy link

0crat commented Sep 30, 2024

@Yegorov Thank you for your contribution! Your effort is appreciated, but we need to address a few points. You've earned +4 points: +4 as a base reward, but -4 for having only 2 comments during review, which is below our policy's threshold of 6 comments. We encourage more detailed code reviews to improve overall quality. Please continue to contribute and engage in thorough code reviews. Your current balance stands at +168.

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.

5 participants