A couple of bugfixes for the Sonar plugin #2

merged 4 commits into from Feb 18, 2012


None yet

2 participants


Hi George,

I use Intellij and Sonar at work and I was really happy to find a developing plugin to integrate them.

I have made a couple of minor fixes in my branch. Without them I could not start using the plugin:

  • working login/password authentication (btw, char[].toString() method returns rubbish, don't use it)
  • fixed bug with the "busy" animation hiding too early
  • fixed NPE, when no line number is returned for a violation

I would like to contribute those changes to the main branch. Let me know if you have any questions/comments on them. I will continue my work on the plugin as it will definitely be useful for me and my team at work.

Best Regards,


Thanks for the bugfixes Max. I appreciate any feedback and it's good to know that it's being used.

@gshakhn gshakhn merged commit 8873797 into gshakhn:master Feb 18, 2012

@maxtomin Thank you for the contribution to the plugin!

This is a bit late, but I've meant to license this project with LGPL to match the regular sonar project. Before doing this, I wanted to make sure it's ok with you since you contributed some code. It probably doesn't matter with such a small project, but I'd rather check before changing the license.

I should have handled this a while ago, so I apologize for the late notice.

Would you be ok with me licensing the code under LGPL?


Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment