-
-
Notifications
You must be signed in to change notification settings - Fork 55
Container.remove + tests #118
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
Conversation
Job #118 is now in scope, role is |
Pull Request Test Coverage Report for Build 204
💛 - Coveralls |
This pull request #118 is assigned to @llorllale/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @amihaiemil/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer |
@llorllale/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please. |
@amihaiemil looks good to me |
@rultor merge please |
@llorllale Thanks for your request. @amihaiemil Please confirm this. |
@rultor merge |
@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here |
@amihaiemil Oops, I failed. You can see the full log here (spent 3min)
|
@amihaiemil looks like you need to update your branch |
Order was finished: +15 point(s) just awarded to @llorllale/z |
The job #118 is now out of scope |
Payment to |
PR for #117