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

MulticolorLayout.java:84-85: This class is still handling... #75

Closed
dmarkov opened this issue Feb 4, 2016 · 11 comments
Closed

MulticolorLayout.java:84-85: This class is still handling... #75

dmarkov opened this issue Feb 4, 2016 · 11 comments
Assignees

Comments

@dmarkov
Copy link

dmarkov commented Feb 4, 2016

Puzzle 59-498fab78 in src/main/java/com/jcabi/log/MulticolorLayout.java:84-85 has to be resolved: This class is still handling multiple responsibilities like formatting, parsing and coloring and should be refactored.... The puzzle was created by Igor Piddubnyi on 29-Jan-2016.

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.

@dmarkov
Copy link
Author

dmarkov commented Feb 12, 2016

@jrdalpra it's your task, go ahead

@jrdalpra
Copy link
Member

@dmarkov ok

@dmarkov
Copy link
Author

dmarkov commented Feb 16, 2016

@dmarkov ok

@jrdalpra thanks

@jrdalpra
Copy link
Member

@dmarkov the PR #76 is ready for review. Thanks.

@jrdalpra
Copy link
Member

@dmarkov any reviewer for the PR #76 ? thanks.

@dmarkov
Copy link
Author

dmarkov commented Feb 23, 2016

@dmarkov the PR #76 is ready for review. Thanks.

@jrdalpra thanks, will take a look

@jrdalpra
Copy link
Member

@dmarkov the PR #76 was merged. Thanks.

@dmarkov
Copy link
Author

dmarkov commented Feb 29, 2016

@dmarkov any reviewer for the PR #76 ? thanks.

@jrdalpra still waiting to find the right one

@dmarkov
Copy link
Author

dmarkov commented Feb 29, 2016

@dmarkov the PR #76 was merged. Thanks.

@jrdalpra thanks for that

@dmarkov
Copy link
Author

dmarkov commented Mar 1, 2016

@yegor256 The issue is closed

@dmarkov dmarkov closed this as completed Mar 1, 2016
@dmarkov
Copy link
Author

dmarkov commented Mar 1, 2016

@jrdalpra done, I added 30 mins in payment AP-9J343123PV982513N, 409 hours and 8 mins spent to complete this. +30 added to your rating, current score is: +248

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

2 participants