I've been browsing a bunch of modules lately, and can't help but see that many of them don't follow "best practices" (heh, not that I always do either :)
But, at the very least, shouldn't they be using strict, warnings and lexically scoped variables. CGI and GD are prime examples (CGI appears to haphazardly use my and -w though).
Since module documentation is where a lot of people first learn how to code, shouldn't we be politely asking module owners to update their documentation to provide at least the basics of good practice?
Or am I opening a can of worms that should be kept shut? ;-)