Honestly I think we should do the following: - Take sonar serious and look after that stuff Reported there - Disable the build breaker for some time so it doesn't flag everything as a failed build - Think about how we can avoid ending in the same situation again
If you look at other projects usually you have a build breaker/guard on a PR and then it is the duty of the author to ensure that the code added doesn't break the sonar rules before merging. Due to the fact that we don't follow that it is easy to slip stuff into develop which leads then to a state we have now. Not 100% sure what is best to do here. Sebastian On 2024/03/14 21:27:45 Christofer Dutz wrote: > Hi all, > > So, we have been having failing builds for … well … it feels like … ever. > Because the Sonarcloud check always keeps on marking every commit as Errror. > > I think: Either we pay attention to it, or we disable it. > > So, what do you folks think? Keeping it on and paying attention to it will be > a lot of work, but it would be worth it. > > Chris >
