Tell me more ×
Programmers Stack Exchange is a question and answer site for professional programmers interested in conceptual questions about software development. It's 100% free, no registration required.

We are hoping to leverage a static code analysis tool (Sonar) as part of our continuous integration server, and are hoping to determine some useful guidelines to serve as a base for allowing the deployment to continue. What conditions should we make mandatory before allowing a build to proceed to the next set of testing?

The obvious answers include that it compiles and the unit tests are successful. But what are some other things we should require before allowing a build to not be rolled back?

share|improve this question
1  
Voting to close as not constructive because, as your own answers illustrate, this is more of a poll than a question with a single correct answer (or at least a set of similar answers). – delnan Oct 9 '12 at 17:49
1  
I've also flagged this as not constructive, this isn't really suitable for the site. However if you are interested in expanding it, it could make for a great blog post for the Programmers Blog, the about page there describes how you can contribute. – Roc Martí Oct 9 '12 at 23:47

closed as not constructive by delnan, Robert Harvey, Yannis Rizos Oct 10 '12 at 0:37

As it currently stands, this question is not a good fit for our Q&A format. We expect answers to be supported by facts, references, or specific expertise, but this question will likely solicit debate, arguments, polling, or extended discussion. If you feel that this question can be improved and possibly reopened, see the FAQ for guidance.

3 Answers

No decrease in public API comments.

share|improve this answer
1  
Comments aren't good. Good comments are good. Bad comments are awful, and useless comments are bad because they waste time and space and give a false sense of quality to those who do not realize how useless they are. – delnan Oct 9 '12 at 17:52

No increase in code duplication.

share|improve this answer

No new major, blocker or critical code standard violations.

share|improve this answer

Not the answer you're looking for? Browse other questions tagged or ask your own question.