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

DyTalks.java:285-286: Remove this method and PMD.TooManyMethods... #607

Closed
alex-palevsky opened this issue Sep 30, 2014 · 10 comments
Closed

Comments

@alex-palevsky
Copy link
Contributor

Puzzle 536-6896859a in src/main/java/com/rultor/dynamo/DyTalks.java:285-286 has to be resolved: Remove this method and PMD.TooManyMethods when jcabi-aspects#91 is resolved.

@aistomin
Copy link
Contributor

aistomin commented Oct 1, 2014

@alex-palevsky @yegor256 "when jcabi-aspects#91 is resolved." - jcabi-aspects#91 wasn't resolved. This issue is blocked by this: jcabi/jcabi-aspects#91.

@yegor256
Copy link
Owner

yegor256 commented Oct 1, 2014

@alex-palevsky looks like we need more time here

@alex-palevsky
Copy link
Contributor Author

@alex-palevsky looks like we need more time here

@yegor256 yes, take your time, thanks for letting me know

@alex-palevsky
Copy link
Contributor Author

@aistomin please go ahead, it's your task now, keep this in mind, and don't hesitate to ask any technical questions you may have

@alex-palevsky
Copy link
Contributor Author

@aistomin This task's budget is 30 mins. This is exactly how much will be paid when the problem explained above is solved. See this for more information

@aistomin
Copy link
Contributor

aistomin commented Oct 5, 2014

@alex-palevsky This issue implementation is here: #620 Could you find somebody to review this changes?

@aistomin
Copy link
Contributor

@yegor256 @alex-palevsky We still waiting for review: #620

@aistomin
Copy link
Contributor

@alex-palevsky Pull request was successfully merged: #620. Please, close this issue if everything is ok.

@alex-palevsky
Copy link
Contributor Author

Closing..

@alex-palevsky
Copy link
Contributor Author

@aistomin Much appreciated! 30 mins was added to your account, payment ID 8972862

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

No branches or pull requests

3 participants