Installing cython is not necessary in the starting Dockerfile provided for sum.py

carpentries-incubator / docker-introduction

Reproducible Computational Environments using Containers

https://carpentries-incubator.github.io/docker-introduction/

Other

44 stars 48 forks source link

Installing cython is not necessary in the starting Dockerfile provided for sum.py #213

Closed krishnakumarg1984 closed 4 months ago

krishnakumarg1984 commented 4 months ago

The sum.py python script does not make use of the cython library. Hence, the

RUN pip install cython

line is not necessary and can be deleted from the provided starter Dockerfile. This also helps with minimising the changes due to the breaking behaviour introduced in recent versions of pip that prevents the installation of cython without a forcing flag (see #209 for details).