-
Notifications
You must be signed in to change notification settings - Fork 0
feat: Configure devcontainer #10
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Only small nits
.devcontainer/devcontainer.json
Outdated
"--cap-add=SYS_PTRACE", "--security-opt", "seccomp=unconfined", | ||
|
||
// Use same network as created for docker-compose that create git-server container | ||
"--network=test-git-server_default" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Strange name for VSC network xD
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Totally agree, I'll update in the next commit. Thanks mate 😄
.devcontainer/devcontainer.json
Outdated
|
||
// Add the IDs of extensions you want installed when the container is created in the array below. | ||
"extensions": [ | ||
"golang.go" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are installing docker, maybe you should add here docker extension. WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think is a good idea, added in the next commit 😄
…iner created by devcontainer
.devcontainer