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

SSH.java:72-74: Refactor this class into smaller... #50

Closed
dmarkov opened this issue Mar 30, 2016 · 13 comments
Closed

SSH.java:72-74: Refactor this class into smaller... #50

dmarkov opened this issue Mar 30, 2016 · 13 comments
Labels

Comments

@dmarkov
Copy link

dmarkov commented Mar 30, 2016

Puzzle 30-15484dec in src/main/java/com/jcabi/ssh/SSH.java:72-74 has to be resolved: Refactor this class into smaller ones to avoid null checking of passphrase. There should probably be separate classes for encrypted/unencrypted private key.... The puzzle was created by Martin Macko on 16-Mar-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 Apr 4, 2016

@NikolaDespotoski the task is yours, go ahead

@dmarkov
Copy link
Author

dmarkov commented Apr 20, 2016

@NikolaDespotoski 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

do not forget to remove the TODO puzzle from the source code

-30 to your rating, your total score is -98

@NikolaDespotoski
Copy link

@dmarkov assign someone else

@dmarkov
Copy link
Author

dmarkov commented Apr 22, 2016

@dmarkov assign someone else

@NikolaDespotoski got it, someone else will be assigned soon

@dmarkov
Copy link
Author

dmarkov commented Aug 8, 2016

@amihaiemil the issue is yours, please help

@amihaiemil
Copy link
Member

amihaiemil commented Aug 9, 2016

@dmarkov I created PR #52 for this

@dmarkov
Copy link
Author

dmarkov commented Aug 9, 2016

@dmarkov I created PR #52 for this

@amihaiemil thank you

@amihaiemil
Copy link
Member

@dmarkov I need more time

@dmarkov
Copy link
Author

dmarkov commented Aug 22, 2016

@dmarkov I need more time

@amihaiemil OK, no problem, take your time

@amihaiemil
Copy link
Member

@dmarkov assign someone else

@amihaiemil amihaiemil removed their assignment Nov 22, 2016
@dmarkov
Copy link
Author

dmarkov commented Nov 22, 2016

@dmarkov assign someone else

@amihaiemil 30 points deducted from your rating

@dmarkov
Copy link
Author

dmarkov commented Nov 22, 2016

@dmarkov assign someone else

@amihaiemil I will assign somebody else to this issue

@dmarkov
Copy link
Author

dmarkov commented Dec 22, 2016

@yegor256 I am closing it, thanks all

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

No branches or pull requests

3 participants