Updated Dockerfile for wider compatibility. Updated readme to use docker image in one command.#1624
Updated Dockerfile for wider compatibility. Updated readme to use docker image in one command.#1624smit-io wants to merge 1 commit intotonsky:masterfrom
Conversation
…ker image in one command.
|
I couldn't get the docker container to run, tried to build one using the Dockerfile but it wouldn't work. |
|
Just tried on fresh checkout of Fira Code -- it still works for me. What was the error you were seeing? |
|
I can't get the image to build using the Dockerfile. I figured this is because of |
|
I got the same error message when using docker to customize Fira Code.
|
|
TLDR: In the first line of "FiraCode-master/Dockerfile", change I figured out that in the "FiraCode-master/Dockerfile" the author used |
|
Maybe I don’t understand the idea of Docker, but why are you trying to build your own image instead of pulling the one that is already built? Isn’t it how reproducible builds work? |
|
Got it! I’m still a Docker newbie and thought |
I just updated the dockerfile because the one that is available can't produce the same docker image again. By including dependencies and specific python version, the dockerfile is made more robust and won't break if new python versions are released. Currently it does. So, I thought of creating a PR. |
|
Yeah, locking down Docker file makes sense of course, for the time I need to build a new one. I’ll look into it |
No description provided.