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

#345: make popular options transmission directly to YAML configuration #396

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

Yegorov
Copy link
Contributor

@Yegorov Yegorov commented Oct 16, 2024

Closes #345

@Yegorov
Copy link
Contributor Author

Yegorov commented Oct 16, 2024

@Suban05 take a look, please

@Yegorov
Copy link
Contributor Author

Yegorov commented Oct 16, 2024

@yegor256 please check

@yegor256 yegor256 merged commit beefd60 into zerocracy:master Oct 24, 2024
8 checks passed
@yegor256
Copy link
Member

@Yegorov thanks!

@0crat
Copy link

0crat commented Oct 24, 2024

@Suban05 Great job on the review! 🌟 You've earned +4 points: +4 base, -10 for limited comments (2), but +10 to ensure recognition. Your current balance stands at +60. Keep up the good work and aim for more detailed feedback next time!

@0crat
Copy link

0crat commented Oct 24, 2024

@Yegorov Hey there! 👋 Nice work on your contribution! You've snagged +4 points for this one. Here's the breakdown: +4 base points, -4 because we only got 2 comments (we're aiming for at least 6), and +4 to keep you in the game. Remember, more hits-of-code and active code reviews can boost your points next time. Keep the contributions flowing! Your current balance is +86. Let's aim higher together! 💪

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.

YAML configuration is too complex
4 participants