-
-
Notifications
You must be signed in to change notification settings - Fork 55
For #170: Added tests to Images.import #249
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 #249 is now in scope, role is |
This pull request #249 is assigned to @bkuzmic/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; there will be no monetary reward for this job |
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.
@paulodamaso Thanks for the PR. Looks good!
@rultor merge it. |
@bkuzmic 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 @paulodamaso Oops, I failed. You can see the full log here (spent 3min)
|
@rultor merge |
@amihaiemil @paulodamaso The pull request is closed already, so I can't merge it |
@amihaiemil I'm sorry, I don't understand you :( Check this page and try again please |
Job was finished in 26 hours, bonus for fast delivery is possible (see §36) |
The job #249 is now out of scope |
Payment to |
For #170: