Commented-out tests?
Nate Finch
nate.finch at canonical.com
Fri Aug 29 19:55:55 UTC 2014
Git blame says ask Roger ;)
100% agree... don't leave in commented out code.
If you feel you must, because you expect it to get uncommented very soon,
then leave a very clear comment about why it was commented out and when it
should get uncommented.
But mostly just don't.
On Fri, Aug 29, 2014 at 3:28 PM, Katherine Cox-Buday <
katherine.cox-buday at canonical.com> wrote:
> Hey all,
>
> I ran into some commented out tests while making a change:
> https://github.com/juju/juju/pull/630/files#r16874739
>
> I deleted them since keeping things around that we might need later is the
> job of source control, not comments ;)
>
> Ian just wanted me to check just to make sure this was OK. If not, please
> chime in on the PR ASAP.
>
> Thanks all!
>
> -
> Katherine
>
> --
> Juju-dev mailing list
> Juju-dev at lists.ubuntu.com
> Modify settings or unsubscribe at:
> https://lists.ubuntu.com/mailman/listinfo/juju-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.ubuntu.com/archives/juju-dev/attachments/20140829/d1b01644/attachment.html>
More information about the Juju-dev
mailing list