Skip to content
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

TeeInputFromBytesTest.java:41-43: Create a new Matcher... #721

Closed
0pdd opened this issue Feb 28, 2018 · 18 comments
Closed

TeeInputFromBytesTest.java:41-43: Create a new Matcher... #721

0pdd opened this issue Feb 28, 2018 · 18 comments

Comments

@0pdd
Copy link
Collaborator

0pdd commented Feb 28, 2018

The puzzle 631-845093d4 from #631 has to be resolved:

* @todo #631:30min Create a new Matcher that will compare results of TeeInput
* as well as copied content to the original message. Apply this new matcher
* for this test and for {@link TeeInputFromByteArrayTest}.

The puzzle was created by Roman Proshin on 26-Feb-18.

Estimate: 30 minutes, role: DEV.

If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

@0crat
Copy link
Collaborator

0crat commented Feb 28, 2018

@yegor256/z please, pay attention to this issue

@0crat 0crat added the scope label Feb 28, 2018
@0crat
Copy link
Collaborator

0crat commented Feb 28, 2018

Job #721 is now in scope, role is DEV

@proshin-roman
Copy link
Contributor

I've already implemented this matcher but had to delete that. I've published my work here so you - whoever will fix this bug - can use it (if it's still acceptable).

@0crat
Copy link
Collaborator

0crat commented Mar 8, 2018

The job #721 assigned to @proshin-roman/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

@proshin-roman
Copy link
Contributor

@neonailol I've created a pull request #731 for this ticket - please review it.

@proshin-roman
Copy link
Contributor

@neonailol ping

@0crat
Copy link
Collaborator

0crat commented Mar 16, 2018

@proshin-roman/z this job was assigned to you 8 days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

@0crat
Copy link
Collaborator

0crat commented Mar 18, 2018

The user @proshin-roman/z resigned from #721, please stop working. Reason for job resignation: It is older than 10 days, see §8

@0crat
Copy link
Collaborator

0crat commented Mar 18, 2018

Resigned on delay, see §8: -30 points just awarded to @proshin-roman/z

@0crat
Copy link
Collaborator

0crat commented Mar 18, 2018

The job #721 assigned to @flslkxtc/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

@0pdd
Copy link
Collaborator Author

0pdd commented Mar 20, 2018

The puzzle 631-845093d4 has disappeared from the source code, that's why I closed this issue.

@0crat
Copy link
Collaborator

0crat commented Mar 20, 2018

@elenavolokhova/z please review this job completed by @flslkxtc/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 Mar 20, 2018
@0crat
Copy link
Collaborator

0crat commented Mar 20, 2018

The job #721 is now out of scope

@proshin-roman
Copy link
Contributor

@yegor256 the issue was fixed by my pull request #731 but the tissue itself was already assigned to another developer. Can it be assigned back to me now (as an author of the fix)? Or what's the correct solution according to Policy?

@yegor256
Copy link
Owner

yegor256 commented Mar 20, 2018

@proshin-roman according to the policy, you lost this job two days ago, because of delay. Also, the quality here will be "bad," since the new assignee didn't do the job -- QA will spot this.

@elenavolokhova
Copy link

@0crat quality bad

@0crat
Copy link
Collaborator

0crat commented Mar 20, 2018

Quality is low, no payment, see §31

@0crat
Copy link
Collaborator

0crat commented Mar 20, 2018

Quality review completed: +8 points just awarded to @elenavolokhova/z

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

No branches or pull requests

5 participants