[development] Trust (was: Hit and run contrib)
Eric-Alexander Schaefer
eric.schaefer at eas-consulting.de
Sat May 31 15:44:17 UTC 2008
Jakob Petsovits schrieb:
> On the other hand, just recently I noticed a missing break; statement
> somewhere in Views. Checked out with CVS. But it didn't directly concern me
> or pose any problems to what I was doing, and opening up a new issue would
> have taken more time and effort than I was willing to put in. Like, "someone
> will probably fix it anyways". With commit access, I would have committed the
> fix right away, and in the unlikely case that it's actually wrong (and a
> "// fall through" comment would be required instead) then Earl would have
> noticed and fixed it the right way. It's probably still unfixed, I guess.
Would you fix the bug without opening an issue which documents it? That
might be OK if you are on your own. But its a really bad idea in a team
environment.
Views is a large module and if you do not grok it, how do you know the
break isn't missing on purpose?
Eric
More information about the development
mailing list