RqHref.java:83-86: RqMethod already validates Request-Line and...

[davvd]

Puzzle 445-5323c196 in src/main/java/org/takes/rq/RqHref.java:83-86 has to be resolved: RqMethod already validates Request-Line and extracts HTTP Method from it. We should extract all important information from Request-Line (HTTP method, URI and HTTP version) in one place to enforce DRY principle.... The puzzle was created by Ikar Sokolov on 15-Dec-2015.

If you have any technical questions, don't ask me, submit new tickets instead

[davvd]

@xupyprmv please go ahead, it's your task now, keep this in mind, and don't hesitate to ask any technical questions you may have. Budget here is 30 mins (keep this principle in mind)

[davvd]

@yegor256 there is no milestone yet, so I set it to 1.0

[davvd]

@xupyprmv you're holding this ticket for 15 days already. If it is not finished (and closed) in the next 48 hours, it will be re-assigned to someone else, see No Obligations principle (this article should help if you're stuck). as a reminder, don't forget to remove the @todo puzzle from the source code. -30 added to your rating, at the moment it is: -30

[xupyprmv]

@davvd The PR #484 is on CR by @yegor256. It's not my fault.

[yegor256]

@xupyprmv everything is your fault, until the task is done. this is our philosophy. when you see that the task is delayed, do something about it. ask @davvd, or me, but don't just wait. we're not managing you, you're managing yourself. I would recommend to read the policy again: http://at.teamed.io/policy

[davvd]

@davvd The PR #484 is on CR by @yegor256. It's not my fault.

@xupyprmv everything is your fault while you're working with a ticket, see http://at.teamed.io/policy.html#7

[xupyprmv]

@davvd I need more time for that ticket. PR #484 is on CR and it will be merged soon, because last CR result doesn't contain any significant remarks.

[xupyprmv]

@davvd This issue is fixed. PR #484 was merged. Please close this ticket if you have no more remarks.

[davvd]

@davvd I need more time for that ticket. PR #484 is on CR and it will be merged soon, because last CR result doesn't contain any significant remarks.

@xupyprmv agreed, take your time

[davvd]

@davvd This issue is fixed. PR #484 was merged. Please close this ticket if you have no more remarks.

@xupyprmv sure, will do right after the next deployment cycle

[davvd]

@yegor256 The task is closed, thanks everybody

[davvd]

@ypshenychka please, help us to evaluate quality here, as per par.24

[ypshenychka]

@davvd Quality is good.

[davvd]

@davvd Quality is good.

@ypshenychka thank you

[davvd]

@xupyprmv I added 10 mins to @ypshenychka (for QA review) in transaction 74072046; Much appreciated! 30 mins was added to your account, payment ID AP-3SK1209573502924F, time spent is 458 hours and 56 mins; added +30 to your rating, now it is equal to +15