-
Notifications
You must be signed in to change notification settings - Fork 127
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
fix(#3357): moved snippet test to integration testing #3361
fix(#3357): moved snippet test to integration testing #3361
Conversation
@yegor256 let's try to merge this with rultor |
@rultor merge |
@maxonfjvipon @yegor256 Oops, I failed. You can see the full log here (spent 10min)
|
@yegor256 let's try again, should work |
@rultor merge |
@maxonfjvipon hurray! thanks! |
@maxonfjvipon Hey there! 👋 Great job on your contribution! You've earned +4 points for this one. Here's the breakdown: +4 base points, +7 for your 142 hits-of-code (nice work!), but -16 for missing the code review (oops!). We bumped it up by 9 to keep you motivated. Remember, code reviews are crucial - they can boost your points and improve code quality. Keep the contributions coming! Your current balance is +68. Let's aim for more points next time! 💪 |
Ref: #3357
PR-Codex overview
This PR updates test files and configurations, changes aliases in a list, and refactors test classes.
Detailed summary