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

HeadInput #764

Closed
yegor256 opened this issue Apr 9, 2018 · 27 comments
Closed

HeadInput #764

yegor256 opened this issue Apr 9, 2018 · 27 comments

Comments

@yegor256
Copy link
Owner

yegor256 commented Apr 9, 2018

Let's create HeadInput class which will behave like an Input, but only showing the first N bytes of the stream:

new HeadInput(new InputOf(new File("very-log-file.bin")));
@0crat 0crat removed their assignment Apr 9, 2018
@0crat 0crat added the scope label Apr 9, 2018
@0crat
Copy link
Collaborator

0crat commented Apr 9, 2018

Job #764 is now in scope, role is DEV

@0crat
Copy link
Collaborator

0crat commented Apr 9, 2018

Bug was reported, see §29: +15 point(s) just awarded to @yegor256/z

@0crat
Copy link
Collaborator

0crat commented Apr 14, 2018

The job #764 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

memoyil added a commit to memoyil/cactoos that referenced this issue Apr 15, 2018
@0crat
Copy link
Collaborator

0crat commented Apr 19, 2018

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

@smyachenkov
Copy link
Contributor

@0crat refuse

@0crat
Copy link
Collaborator

0crat commented Apr 21, 2018

@0crat refuse (here)

@flslkxtc The user @flslkxtc/z resigned from #764, please stop working. Reason for job resignation: Order was cancelled

@0crat
Copy link
Collaborator

0crat commented Apr 21, 2018

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @flslkxtc/z

@0crat
Copy link
Collaborator

0crat commented Apr 23, 2018

The job #764 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

@0crat
Copy link
Collaborator

0crat commented Apr 28, 2018

@proshin-roman/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.

@proshin-roman
Copy link
Contributor

proshin-roman commented May 1, 2018

@memoyil why didn't you submit a pull request with your changes? It seems you've already implemented this ticket in your branch.

@memoyil
Copy link

memoyil commented May 1, 2018

@proshin-roman I will create a new pull request after refactoring my code.

@proshin-roman
Copy link
Contributor

@memoyil The thing is that the ticket is assigned to me, so I have to submit some fix for it. I don't want to steal your work, but I have to implement something similar. The questions are why didn't you assign the ticket to yourself, and why did you start working on a ticket that is not assigned to you?

@memoyil
Copy link

memoyil commented May 1, 2018

@proshin-roman You are correct. I also don't want to hijack your assigned task. Actually at first when nobody was assigned to that task I started to work on it. My zerocracy account creation still pending that's why tasks could not be assigned to me.

@proshin-roman
Copy link
Contributor

@memoyil ok, I'm going to submit some minimal solution that will fit the requirements of the ticket, but you still can submit your constructors and tests later. I think it should be ok.

@proshin-roman
Copy link
Contributor

@0crat wait until pull request #801 is reviewed

@0crat
Copy link
Collaborator

0crat commented May 1, 2018

@0crat wait until pull request #801 is reviewed (here)

@proshin-roman The impediment for #764 was registered successfully by @proshin-roman/z

@proshin-roman
Copy link
Contributor

@yegor256 The feature has been implemented in #801 that is already merged into the master branch. Please, check if the ticket can be closed.

@proshin-roman
Copy link
Contributor

@yegor256 Please, close the ticket.

@proshin-roman
Copy link
Contributor

@llorllale maybe you can close the ticket?

@llorllale
Copy link
Contributor

@proshin-roman sorry, no, @yegor256 is active on the platform

@llorllale
Copy link
Contributor

@yegor256 ping

@yegor256
Copy link
Owner Author

@proshin-roman thanks!

@0crat
Copy link
Collaborator

0crat commented May 10, 2018

@elenavolokhova/z please review this job completed by @proshin-roman/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 May 10, 2018
@0crat
Copy link
Collaborator

0crat commented May 10, 2018

The job #764 is now out of scope

@elenavolokhova
Copy link

@0crat quality good

@0crat
Copy link
Collaborator

0crat commented May 10, 2018

Order was finished, quality is "good": +35 point(s) just awarded to @proshin-roman/z

@0crat
Copy link
Collaborator

0crat commented May 10, 2018

Quality review completed: +8 point(s) 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

7 participants