There are some wrong/out of date configurations in the Dockerfiles under .devcontainers.
All the base images in these Dockerfiles (python:3.10 and python:3.11-sim-bookworm) can already install the git-lfs through apt-get directly without adding these manually. Therefore this line is actually out of date
# add git lhs to apt
RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | bash
(Also, the typo here is not totally fixed. There's still typo in the comment)
The base image, python:3.11-slim-bookworm, which is used in dev and full Dockerfiles, does not comes with curl ready to use, which means that this configuration for these two images
# add git lhs to apt
RUN curl -s https://packagecloud.io/install/repositories/github/git-lfs/script.deb.sh | bash
actually fails silently (the output was redirected, which I think could be the reason). This made me very confused when I was trying to modify the Dockerfile and I think this one should be deleted.
Also, I'm not quite understand why different base images are used for different versions. Is there any reason behind this?
Describe the issue
There are some wrong/out of date configurations in the Dockerfiles under .devcontainers.
python:3.10andpython:3.11-sim-bookworm) can already install thegit-lfsthroughapt-getdirectly without adding these manually. Therefore this line is actually out of date(Also, the typo here is not totally fixed. There's still typo in the comment)
python:3.11-slim-bookworm, which is used indevandfullDockerfiles, does not comes withcurlready to use, which means that this configuration for these two imagesactually fails silently (the output was redirected, which I think could be the reason). This made me very confused when I was trying to modify the Dockerfile and I think this one should be deleted.
Steps to reproduce
No response
Screenshots and logs
No response
Additional Information
No response