Closed marcitqualab closed 5 months ago
Looks good.
There is of course no precise answer what "minimal" means.
Just 2 comments:
In this line you hardcode a version of clj-new
..
This means "conceptually", that each new version of clj-new
requires an update of the template.
(If you do them or not, is then an other question)
Regarding sshd
, I would prefer to remove it, as this is "security" related.
I would prefer not to have a default devcontainer config which adds an accessible ssh server.
So please remove this:
https://github.com/scicloj/devcontainer-templates/blob/451edf34041056fd5b70b9bdf4f154e70b432fee/src/basecloj/.devcontainer/devcontainer.json#L19
(does this related to https://github.com/scicloj/devcontainer-templates/blob/451edf34041056fd5b70b9bdf4f154e70b432fee/src/basecloj/.devcontainer/devcontainer.json#L12 ? , if yes please remove as well)
Hello @behrica ,
Yes the clj-new version for now is hardcoded. The official readme file recommends this way to install it. I have created a task on my side to research how to auto-update this version over time using the github actions.
I have removed the ssh and security options.
Many thanks,
Changes looks good. Do you still want to make the image tag configurable ? as I commented on this file : src/basecloj/.devcontainer/Dockerfile
Hello @behrica ,
I do not see any specific comment in the file : src/basecloj/.devcontainer/Dockerfile.
If you are referring to the clj-new version tag, I think that for now we can start with the current latest version as it is and I will research and test how to implement an auto update or use latest tag if possible.
I can create a new PR later on when I have an update on how to remove the hard coded clj-new version tag.
Would that work for you?
No ,I talk about the "tag" of the base image in this line in teh Dockerfile:
FROM clojure:temurin-21-tools-deps-jammy
you have hardcoded the tag of the image, namely:
temurin-21-tools-deps-jammy
But there are lots of others: https://hub.docker.com/_/clojure/tags
And I think we should allow the user to "choose" when using the template. This can be done via "template variables". If we have those, VSCode renders the options and lets the user pick one, the moment he is using the template.
The question is then:
See here https://github.com/scicloj/devcontainer-templates/blob/main/src/scicloj/devcontainer-template.json what I think is the best solution for the "which options", (only LTS and JDK as supported by Clojure). In any case a user can override it after applying the template. But In VSCode he will only "see" the options we explicitly list.
Ah, yes, got it. I will create some options then. Thanks.
Hello, this is the first version of the baseline clojure template.