-
Notifications
You must be signed in to change notification settings - Fork 33
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
Removed puzzle for issue #78 #81
Conversation
@rultor merge |
@amihaiemil @yegor256 Oops, I failed. You can see the full log here (spent 6min)
|
@yegor256 apparently rultor does not have rights to access Maven (http://repo.maven.apache.org/maven2) ? The same exception were thrown when the previous PR was deployes by Dan. Is there something wrong on rultor's side? |
@amihaiemil I created #82 explaining what's going on here. This is something that needs to be fixed here not in Rultor I fear. |
@original-brownbear I read your issue, but I must admit I don't understand what needs to be done (I have 0 experience with dev-ops stuff like Docker... for instance what do you mean when you say "image"?) I have the feeling that it's a 5 minutes job for someone who know what they' re doing, while it would take me as a newbie much more time and I would probably break something... I'll ask the PM to put this PR and its issue on hold until issue #82 is solved. Thank you very much for the support. |
@amihaiemil Thanks for your pull request, let me find someone who can review it |
@yegor256 review this pull request please, it is rather short for a full review |
@rultor try to merge again, maybe will work |
@amihaiemil @yegor256 Oops, I failed. You can see the full log here (spent 7min)
|
@rultor try to merge |
@amihaiemil @yegor256 Oops, I failed. You can see the full log here (spent 16s)
|
@yegor256 I saw you fixed the |
@rultor try to merge |
@rultor please deploy |
In the previous PR that solved issue #78 I had forgotten to remove the puzzle.
In this PR I removed the puzzle.