search

mozilla / sugardough

A web application template based on Django.
Apache License 2.0
164 stars 31 forks source link

Wrong chown syntax in the dockerfile #138

Closed maurodoglio closed 7 years ago

maurodoglio commented 7 years ago

The syntax for chown here seems to be wrong, or at the very least is not the most common way to use it. I was expecting it to use a colon between user and group like 

RUN chown webdev:webdev -R .
glogiotatidis commented 7 years ago

Syntax is valid and I agree that's not as common but it's quicker and thus I'm used to it. Happy to merge a PR to change this though if you feel like it. I'll close this issue as it's not a bug.

Thanks! 🍺