-
Notifications
You must be signed in to change notification settings - Fork 90
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
improve docker docs #315
improve docker docs #315
Conversation
Signed-off-by: Agustín Ramiro Díaz <agustin.ramiro.diaz@gmail.com>
✅ Deploy Preview for surrealdb-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
thanks for the PR @AgustinRamiroDiaz. I would suggest using the That way there's no need to change the file ownership |
+1 and the container will be based on. It's the latest-dev version of the surrealdb/surrealdb image. |
@sgirones thanks for the warm welcome :D I understand that the WDYT about adding @Ekwuno sorry, I don't understand your comment. What does "the container will be based on" mean? |
@AgustinRamiroDiaz sorry about that. I mean that adding |
fixes surrealdb/surrealdb#3468
Why
Docker file permissions are hard to understand, and cause trouble for newcomers. This PR adds an extra step to reduce the likelihood of errors for users.
Context
There's more context in the issue
Notes
I'm not a professional documentation creator, modify as you please