-
Notifications
You must be signed in to change notification settings - Fork 11
Conversation
Job #334 is now in scope, role is |
This pull request #334 is assigned to @carlosmiranda/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @yegor256/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 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.
@g4s8 one comment
xml/pmo/resumes/-duplicates.xml
Outdated
</resume> | ||
<resume login="yegor256"> | ||
<submitted>2018-01-01T00:00:00</submitted> | ||
<examiner>g4s8</examiner> |
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.
@g4s8 since we're enforcing uniqueness of the login attribute, let's make the rest of data different.
@carlosmiranda fixed in f921594, please look again |
@yegor256 good to merge |
@carlosmiranda/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. |
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.
@yegor256 ping, good to merge this now
@rultor merge |
@ypshenychka/z please review this job completed by @carlosmiranda/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed |
The job #334 is now out of scope |
@carlosmiranda According to our QA Rules:
Only one issue was found during code review. |
@ypshenychka confirmed |
@carlosmiranda Thanks. |
@0crat quality acceptable |
Order was finished, quality is "acceptable": +15 point(s) just awarded to @carlosmiranda/z |
Quality review completed: +8 point(s) just awarded to @ypshenychka/z |
zerocracy/farm#800 - resumes schema, tests, xsl