Skip to content

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

Merged
merged 7 commits into from
Jan 31, 2022
Merged

Conversation

xoanmm
Copy link
Contributor

@xoanmm xoanmm commented Jan 24, 2022

  • Added necessary files for use devcontainer under directory .devcontainer
  • Remove innecesary files

@xoanmm xoanmm self-assigned this Jan 24, 2022
@xoanmm xoanmm requested a review from JorTurFer January 24, 2022 14:21
Copy link
Contributor

@JorTurFer JorTurFer left a 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

"--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"
Copy link
Contributor

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

Copy link
Contributor Author

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 😄


// Add the IDs of extensions you want installed when the container is created in the array below.
"extensions": [
"golang.go"
Copy link
Contributor

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?

Copy link
Contributor Author

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 😄

@xoanmm xoanmm changed the title feat: improve user experience [SRE-1299] - feat: improve user experience Jan 31, 2022
@xoanmm xoanmm changed the title [SRE-1299] - feat: improve user experience [SRE-1299] - Configure devcontainer Jan 31, 2022
@xoanmm xoanmm changed the title [SRE-1299] - Configure devcontainer [SRE-1299] feat: Configure devcontainer Jan 31, 2022
@xoanmm xoanmm changed the title [SRE-1299] feat: Configure devcontainer feat: Configure devcontainer Jan 31, 2022
@xoanmm xoanmm merged commit ca3b99c into develop Jan 31, 2022
@xoanmm xoanmm deleted the feature/improve-user-experience branch January 31, 2022 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants