-
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
Migrate to logback & enable slf4j-bridge #558
Conversation
Codecov Report
@@ Coverage Diff @@
## master #558 +/- ##
============================================
+ Coverage 80.83% 81.78% +0.95%
- Complexity 477 481 +4
============================================
Files 133 133
Lines 2478 2471 -7
Branches 220 220
============================================
+ Hits 2003 2021 +18
+ Misses 404 382 -22
+ Partials 71 68 -3
Continue to review full report at Codecov.
|
@rultor merge |
@andreoss @yegor256 Oops, I failed. You can see the full log here (spent 11min)
|
@andreoss what do you suggest we do here? |
@yegor256 Giving more heap space to rultor container should solve this |
@andreoss you are welcome to submit a pull request, making changes to |
@yegor256 What are the container's limits? |
This reverts commit 722be24.
@andreoss this is how Rultor starts Docker: https://github.com/yegor256/rultor/blob/master/src/main/resources/com/rultor/agents/req/_head.sh#L102-L108 It seems that it's 6Gb. |
@rultor merge |
@andreoss @yegor256 Oops, I failed. You can see the full log here (spent 5min)
|
@rultor merge |
@andreoss @yegor256 Oops, I failed. You can see the full log here (spent 4min)
|
@rultor merge |
@andreoss @yegor256 Oops, I failed. You can see the full log here (spent 2min)
|
@andreoss @yegor256 Oops, I failed. You can see the full log here (spent 2min)
|
Per #554