Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

Resumes #334

Merged
merged 2 commits into from
Apr 5, 2018
Merged

Resumes #334

merged 2 commits into from
Apr 5, 2018

Conversation

g4s8
Copy link
Contributor

@g4s8 g4s8 commented Mar 26, 2018

zerocracy/farm#800 - resumes schema, tests, xsl

@0crat 0crat added the scope label Mar 26, 2018
@0crat
Copy link
Collaborator

0crat commented Mar 26, 2018

Job #334 is now in scope, role is REV

@0crat
Copy link
Collaborator

0crat commented Mar 26, 2018

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

Copy link
Contributor

@carlosmiranda carlosmiranda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@g4s8 one comment

</resume>
<resume login="yegor256">
<submitted>2018-01-01T00:00:00</submitted>
<examiner>g4s8</examiner>
Copy link
Contributor

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.

@g4s8
Copy link
Contributor Author

g4s8 commented Mar 29, 2018

@carlosmiranda fixed in f921594, please look again

@carlosmiranda
Copy link
Contributor

@yegor256 good to merge

@0crat
Copy link
Collaborator

0crat commented Mar 31, 2018

@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.

Copy link
Contributor

@carlosmiranda carlosmiranda left a 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

@yegor256
Copy link
Owner

yegor256 commented Apr 5, 2018

@rultor merge

@rultor
Copy link
Contributor

rultor commented Apr 5, 2018

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit f921594 into yegor256:master Apr 5, 2018
@g4s8 g4s8 deleted the farm-800 branch April 5, 2018 13:49
@rultor
Copy link
Contributor

rultor commented Apr 5, 2018

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 3min)

@0crat
Copy link
Collaborator

0crat commented Apr 5, 2018

@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

@0crat 0crat removed the scope label Apr 5, 2018
@0crat
Copy link
Collaborator

0crat commented Apr 5, 2018

The job #334 is now out of scope

@0crat
Copy link
Collaborator

0crat commented Apr 5, 2018

Payment to ARC for a closed pull request, as in §28: +10 point(s) just awarded to @yegor256/z

@ypshenychka
Copy link

@carlosmiranda According to our QA Rules:

The code reviewer found at least three problems in the code.
Comments were mostly about design problems, not cosmetic issues.

Only one issue was found during code review.
Please confirm that you'll try to find at least three problems while future reviews.

@carlosmiranda
Copy link
Contributor

@ypshenychka confirmed

@ypshenychka
Copy link

@carlosmiranda Thanks.

@ypshenychka
Copy link

@0crat quality acceptable

@0crat
Copy link
Collaborator

0crat commented Apr 6, 2018

Order was finished, quality is "acceptable": +15 point(s) just awarded to @carlosmiranda/z

@0crat
Copy link
Collaborator

0crat commented Apr 6, 2018

Quality review completed: +8 point(s) just awarded to @ypshenychka/z

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants