Yeah in general that's what I would prefer also. In this specific instance
I decided to leave it visible since I plan to be uncommenting that code
sometime this weekend.  I most emphatically don't want to start a trend
with this. :-)

On Sat, May 11, 2013 at 10:06 AM, e0d <notifications@github.com> wrote:

> In playbooks/roles/nginx/tasks/main.yml:
>
> >  - name: Create nginx log file location (just in case)
> >    file: path={{log_base_dir}}/nginx state=directory owner=syslog group=adm mode=2770
> >    tags:
> >    - nginx
> >    - logging
> > +
> > +# Commented out until default config has nginx log to {{log_base_dir}}/nginx
>
> Can we stash stuff like this rather than leaving in commented code?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/edx/configuration/pull/57/files#r4183611>
> .
>