-
-
Notifications
You must be signed in to change notification settings - Fork 23
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
Update core to 2.263.3, args4j to 2.33 #21
Conversation
I'm not sure why the PR build on ci.jenkins.io runs out of memory and spends 10 hours running, while the master branch completes in 30 minutes or less. Any insights to offer? |
So the main method checks for I guess the next step is to try it locally with more plugins than the default ~50 and see if I can reproduce the out of memory. |
It turns out that this mock
was the cause of builds running for hours and running out of memory. I originally added it to avoid some warnings in the log, but those are not critical. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me. Thanks for making the change!
Merging since @kwhetstone had approved the changes previously. |
... and change update the mock code to avoid some exceptions