MakingSense / WebApiCore-Seed

MIT License
2 stars 5 forks source link

Sonar integration #37

Open Dongata opened 6 years ago

Dongata commented 6 years ago

Background  

It's a good thing knowing some metrics of your project, so we add an integration with sonar

Changes done  

Demo 

Sonar Page

matiasbeckerle commented 6 years ago

Please DO NOT merge this yet. I want to review what it does. So far, I'm against adding it on a seed project.

AlphaGit commented 6 years ago

In response to @matiasbeckerle's concern (which I agree with), consider adding it as a optional module (something plugabble, so that not every project needs to take this out) OR having a set of instructions that make it easy to add. (For instance, a branch/document with the changes to apply to include it.)

Code generators are an option buuuuut they might be very complicated and I agree if you guys don't want to follow that route.

mravinale commented 6 years ago

I think it adds value as travis does, seems to be not very difficult to remove and we can track bad practices as we develop this project

matiasbeckerle commented 6 years ago

I think it adds value as travis does

Not exactly. Travis does his thing in a much less invasive form.

seems to be not very difficult to remove

Yes but while working on projects this is hard to make it happen. There is always a different priority.

and we can track bad practices as we develop this project

Indeed... but we have other mechanisms to improve ourselves. I believe that a tool like this one should be a decision made by the team. What if they want to use a different one? How they can track what was required so they can delete the code? I remark again that a seed it's just that: the minimum required to make it grow. We should not create a box full of libraries.

Instead, I propose to see what happens with just the minimum. Where do we fail? What was wrong? What we were missing? What was a leftover? Let's experience with the minimum first, then we iterate. I'm not saying we should never use this tool/lib. I'm saying that for now, I believe we should hold on with adding it.