Closed victornoel closed 5 years ago
This pull request #80 is assigned to @olenagerasimova/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @g4s8/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
@olenagerasimova I've pushed a new commit (and force-push/rebased because of gitlint)
@rultor merge
@rultor merge
@olenagerasimova Thanks for your request. @g4s8 Please confirm this.
@rultor merge
@rultor merge
@rultor merge
@g4s8 OK, I'll try to merge now. You can check the progress of the merge here
@rultor merge
@g4s8 @victornoel The pull request is closed already, so I can't merge it
@rultor merge
@g4s8 I'm sorry, I don't understand you :( Check this page and try again please
Code review was too long (9 days), architects (@g4s8) were penalized, see §55
@olenagerasimova You found less than three problems in the PR, please confirm you will try to find at least three problems next time
@ammaratef45 I confirm
@olenagerasimova thanks
@0crat quality acceptable
This is for #42.