bcgov/von-network

Possible redundant Dockerfile directives

Closed this issue · 1 comments

benjsmi commented

Hello!

https://github.com/bcgov/von-network/blob/master/Dockerfile#L36-L38, which is:

RUN pip3 install -U \
    pip \
    setuptools

And https://github.com/bcgov/von-network/blob/master/Dockerfile#L72-L73, which is:

RUN pip3 install --upgrade setuptools
RUN pip3 install pipenv

Seem redundant to me. I will happily contribute a PR that fixes if you agree that they are redundant.

swcurran commented

Sorry there was no reply on this. We've done a lot on the process for this, so closing this. Feel free to submit a PR anytime.