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

IOExceptions with bootstrap #495

Closed
piotrkot opened this issue Jan 3, 2016 · 39 comments
Closed

IOExceptions with bootstrap #495

piotrkot opened this issue Jan 3, 2016 · 39 comments

Comments

@piotrkot
Copy link
Contributor

piotrkot commented Jan 3, 2016

Here is a sample project with 2 IOExceptions being reproduced.
https://github.com/piotrkot/takesbug

@yegor256
Copy link
Owner

yegor256 commented Jan 4, 2016

@piotrkot hm... interesting. thanks for reporting!

@yegor256 yegor256 added the bug label Jan 4, 2016
@davvd
Copy link

davvd commented Jan 4, 2016

@piotrkot this task will get someone's attention soon

@davvd
Copy link

davvd commented Jan 4, 2016

@piotrkot since there is no milestone yet I set it to "1.0"

@davvd davvd added this to the 1.0 milestone Jan 4, 2016
@davvd
Copy link

davvd commented Jan 14, 2016

@rajithd can you please help? Keep in mind this. If you have any technical questions, don't hesitate to ask right here

The cost of this task is 30 mins (this is exactly how much will be paid, not less not more), when the task is done

@davvd davvd added the @rajithd label Jan 14, 2016
@davvd
Copy link

davvd commented Jan 29, 2016

@rajithd check this "no obligations principle".. This task is on your name for at least 15 days. If you can't close it within the next 48 hours I will have to assign someone else to it. This article should help if you're stuck

-30 added to your rating, current score is: -60

@davvd
Copy link

davvd commented Feb 5, 2016

@rajithd the task is overdue, and I have to re-assign it to someone else. Please, stop working with it immediately. In general, we're against overdue tasks, check this page... -60 added to your rating, current score is: -180

@davvd davvd removed the @rajithd label Feb 5, 2016
piotrkot added a commit to piotrkot/takes that referenced this issue Feb 11, 2016
@davvd
Copy link

davvd commented Feb 28, 2016

@essobedo the task is yours please proceed

@essobedo
Copy link
Contributor

@piotrkot As workaround you can use Bug.class.getResource instead of Bug.class.getResourceAsStream.

@essobedo
Copy link
Contributor

@davvd here is my PR, please find a reviewer, thanks in advance.

@davvd
Copy link

davvd commented Feb 29, 2016

@davvd here is my PR, please find a reviewer, thanks in advance.

@essobedo OK

@piotrkot
Copy link
Contributor Author

@essobedo Yes, I know it could work without the stream. IMHO the constructor with inputstream is a mistake.

@essobedo
Copy link
Contributor

@piotrkot I agree with you it is a mistake, have a look to my PR to see how I fix it without removing the constructors with InputStream.

@essobedo
Copy link
Contributor

@davvd the PR has been merged, the ticket and the PR are ready to be reviewed by the QA. Thank you in advance.

@davvd
Copy link

davvd commented Mar 21, 2016

@essobedo it's not yours any more, because it took too long, please stop working with it. added -60 to your rating, now it is equal to +285

@essobedo
Copy link
Contributor

@essobedo it's not yours any more, because it took too long, please stop working with it. added -60 to your rating, now it is equal to +285

@davvd @yegor256 are you kidding me? The PR has been merged already, I did my job already, it was stuck because you did not ask the QA to review. Losing 60 points with still a quota of 2 instead of 5, it is totally unfair, please give me back my points and add me 30 points for the work done

@essobedo
Copy link
Contributor

@davvd @yegor256 I asked you to ask the QA to review the ticket 4 days ago

@yegor256
Copy link
Owner

@essobedo first you have to close the ticket. only after that the QA will jump in. at the moment the problem is still not solved. too early for QA

@yegor256
Copy link
Owner

@essobedo keep in mind that merging a PR doesn't mean solving a problem. you may have 4 PRs for one ticket and still the problem won't be solved.

@essobedo
Copy link
Contributor

@yegor256 what am I supposed to do then if I wait days for other people to do the part of the job? Like here answers, validations and/or QA? I'm sorry but it is really unfair, I worked hard on this one and you know it, I don't believe that I deserve it. Any time I had an answer, I always proposed a new PR couple hours later even Sundays. If you think that I'm not good enough to work with you, just tell me but please don't punish me when I don't deserve it.

@yegor256
Copy link
Owner

@essobedo good intentions are not enough (basically, we don't care much about intentions). Discipline - this is what is more important. All rules are listed here: http://at.teamed.io/policy Read them once again. I'll put this ticket back to you, because you're new to our process. But keep this in mind for the future: we care only about your objective results. A closed ticket is a result. Everything else is just noise.

@yegor256
Copy link
Owner

@davvd pls assign @essobedo back here

@essobedo
Copy link
Contributor

@yegor256 thank you. Another question why do I still have a quota of 2 instead of 5 as described in the policy?

@essobedo
Copy link
Contributor

@yegor256 here I'm supposed to ask for "more time" then right?

@essobedo
Copy link
Contributor

@davvd more time

@davvd
Copy link

davvd commented Mar 22, 2016

@davvd pls assign @essobedo back here

@yegor256 done, @essobedo is assigned to this task, and is already working on it, AFAIK

@davvd
Copy link

davvd commented Mar 22, 2016

@davvd more time

@essobedo no problem, thanks for letting me know

@essobedo
Copy link
Contributor

@yegor256 @davvd what about my 60 points lost, can't I get them back ? I'm new to your process as @yegor256 said, let's say that it was my only and last warning so please give them me back. Thank you in advance for your understanding

@essobedo
Copy link
Contributor

@davvd the ticket is ready to be reviewed by the QA, thank you in advance?
@davvd how many more days do I have when you accept to give "more time" for the task?

@yegor256
Copy link
Owner

@essobedo we can't revert bonus back. the ticket is NOT ready to be reviewed by QA. it will be ready, when it's closed. please, read this entire article: http://www.yegor256.com/2014/11/24/principles-of-bug-tracking.html

@piotrkot
Copy link
Contributor Author

@essobedo I think now it is the time to ask the one who opened the ticket (that is me) to close it.
Just checked the current version of takes (1.0-SNAPSHOT) with my takesbug. The bug is fixed. Thank you very much for your support.

@essobedo
Copy link
Contributor

@piotrkot you are right, thank you very much

@essobedo
Copy link
Contributor

@davvd The ticket is closed so it is ready for the QA

@davvd
Copy link

davvd commented Mar 23, 2016

@davvd The ticket is closed so it is ready for the QA

@essobedo OK

@davvd
Copy link

davvd commented Mar 24, 2016

@ypshenychka please, review this ticket for compliance with our QA rules

@ypshenychka
Copy link

@davvd Quality is good.

@davvd
Copy link

davvd commented Mar 24, 2016

@davvd Quality is good.

@ypshenychka thanks a lot :)

@davvd
Copy link

davvd commented Mar 24, 2016

@essobedo paied 10 mins to @ypshenychka for QA review (payment ID is 81340165). Thank you, I have added 1 hour to your account in payment/transaction "81340181", time consumed: 20 hours and 7 mins. there is a bonus for fast delivery (m=1207). +60 to your rating, your total score is +345

@essobedo
Copy link
Contributor

@davvd many thanks

@davvd
Copy link

davvd commented Mar 25, 2016

@davvd many thanks

@essobedo np :)

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