{"action":"created","issue":{"url":"https://api.github.com/repos/justrelease/justrelease/issues/3","labels_url":"https://api.github.com/repos/justrelease/justrelease/issues/3/labels{/name}","comments_url":"https://api.github.com/repos/justrelease/justrelease/issues/3/comments","events_url":"https://api.github.com/repos/justrelease/justrelease/issues/3/events","html_url":"https://github.com/justrelease/justrelease/issues/3","id":105678159,"number":3,"title":"Code quality improvement tips","user":{"login":"MathieuAA","id":2753381,"avatar_url":"https://avatars.githubusercontent.com/u/2753381?v=3","gravatar_id":"","url":"https://api.github.com/users/MathieuAA","html_url":"https://github.com/MathieuAA","followers_url":"https://api.github.com/users/MathieuAA/followers","following_url":"https://api.github.com/users/MathieuAA/following{/other_user}","gists_url":"https://api.github.com/users/MathieuAA/gists{/gist_id}","starred_url":"https://api.github.com/users/MathieuAA/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MathieuAA/subscriptions","organizations_url":"https://api.github.com/users/MathieuAA/orgs","repos_url":"https://api.github.com/users/MathieuAA/repos","events_url":"https://api.github.com/users/MathieuAA/events{/privacy}","received_events_url":"https://api.github.com/users/MathieuAA/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-09-09T20:29:10Z","updated_at":"2015-10-16T17:47:50Z","closed_at":"2015-10-16T17:47:50Z","body":"Sorry if I'm a bit harsh, but I'm a bit tired.\r\n\r\nI'm not going to point everything that bothers me, just the basic stuff.\r\n\r\nDon't hesitate abusing private fields =) Like [here ](https://github.com/justrelease/justrelease/blob/master/src/main/java/com/justrelease/JustRelease.java#L34) for instance.\r\n\r\nThe code needs some cleaning-up [here](https://github.com/justrelease/justrelease/blob/master/src/main/java/com/justrelease/JustRelease.java#L34), you can either replace it with a `else`, or exit the method (much better).\r\n\r\nYour code needs some more cleaning as the `#release` method is __way__ too complex (too many lines, refactorable code, etc.).\r\n\r\nIn [this](https://github.com/justrelease/justrelease/blob/master/src/main/java/com/justrelease/JustReleaseCLI.java) file, you should not try-catch, and display the help (and then exit with the system helper), just throw it with a nice message (if the user wants help, I guess he/she can type one more line in the shell).\r\n\r\nThe code needs refactoring, just search for the 'cmd' in the same file (a bad word, you should use 'command') word, and try to make the occurrences nearer to each other.\r\n\r\nIn [this](https://github.com/justrelease/justrelease/blob/master/src/main/java/com/justrelease/project/type/AbstractProjectInfo.java) file, `releaseConfig` needs to be hidden!\r\n\r\nIn the MavenProject class, you use a setter on a field belonging to an abstract class extended by MavenProject. Even if it has no visible incidence, it is considered bad practice...\r\n\r\nTo sum up (I didn't go through every file):\r\n  - Don't use the default visibility in concrete ( != interface) classes;\r\n  - Use consistent styling (the `{}` either need to be put after every if/for/etc instructions everywhere, or no where, just be consistent);\r\n  - Use private methods (or delegate) to make your methods lighter.\r\n\r\nI suggest you use Sonar as it can help you with code quality.\r\n\r\nBy the way, your project looks promising, and I'll test it when I can!"},"comment":{"url":"https://api.github.com/repos/justrelease/justrelease/issues/comments/148783541","html_url":"https://github.com/justrelease/justrelease/issues/3#issuecomment-148783541","issue_url":"https://api.github.com/repos/justrelease/justrelease/issues/3","id":148783541,"user":{"login":"mesutcelik","id":823273,"avatar_url":"https://avatars.githubusercontent.com/u/823273?v=3","gravatar_id":"","url":"https://api.github.com/users/mesutcelik","html_url":"https://github.com/mesutcelik","followers_url":"https://api.github.com/users/mesutcelik/followers","following_url":"https://api.github.com/users/mesutcelik/following{/other_user}","gists_url":"https://api.github.com/users/mesutcelik/gists{/gist_id}","starred_url":"https://api.github.com/users/mesutcelik/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mesutcelik/subscriptions","organizations_url":"https://api.github.com/users/mesutcelik/orgs","repos_url":"https://api.github.com/users/mesutcelik/repos","events_url":"https://api.github.com/users/mesutcelik/events{/privacy}","received_events_url":"https://api.github.com/users/mesutcelik/received_events","type":"User","site_admin":false},"created_at":"2015-10-16T17:47:50Z","updated_at":"2015-10-16T17:47:50Z","body":"many thanks. waiting for your feedback."}}