I'm not in favor of "sprocs" but I prefer them over "store procedures". AFAIK, we're not selling anything.

At work we have DTS that simply call an SP or even better, they have a SQL task with "SELECT 1".

But one of the worst things is having to read code written as if formatting was prohibited and the less lines the better.

Other things are:

SQL-86 Joins and JOIN keywords on the right of the tables.