Give the dockerfile some love (1/x)

[bebehei]

Adapts the Dockerfile to Dockerfile "sort multiline" best practices.

Nothing more. Previous experience taught me that's the hardest part to review for a Dockerfile.

Will create additional PRs to address #60 and fix it at the end.

[humitos]

Looks good to me! Thanks for the changes.

I left some minor comments to do and we can merge it.

[humitos]

Maybe it's better to continue with the visual indentation here.

[bebehei]

Sorry, but I don't get this. Do you want me to write it this way?!?

RUN apt-get -y install \
      python-pip \
    && pip install -U \

[humitos]

RUN apt-get -y install \
      python-pip \
   && pip install -U \

or

RUN apt-get -y install \
      python-pip \
      && pip install -U \

or

RUN apt-get -y install \
      python-pip && \
      pip install -U \

I think 1) and 3) are more readable. 1) looks nicer to me ;)

[bebehei]

All in all, you're aligning commands to package arguments. Really?

In the current code, I've aligned the pip command to the apt-get command and moved the rest around it.

But aligning it to the pip package makes no sense to me, sorry.

[humitos]

Oh sorry, you are right. I got confused. Ignore me.

[humitos]

Same here about visual indentation.

[humitos]

Thanks!