Skip to content
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

add devcontainer #5592

Merged
merged 5 commits into from May 10, 2022
Merged

add devcontainer #5592

merged 5 commits into from May 10, 2022

Conversation

joshspicer
Copy link
Contributor

@joshspicer joshspicer commented May 9, 2022

closes #5582

Adds a devcontainer installing go 1.18 for use in GitHub Codespaces and the VS Code Remote-Containers extension.

Based on See here for image contents: https://github.com/microsoft/vscode-dev-containers/tree/v0.234.0/containers/go/.devcontainer/base.Dockerfile

@joshspicer joshspicer requested a review from a team as a code owner May 9, 2022
@joshspicer joshspicer requested review from samcoe (assigned from cli/code-reviewers) and removed request for a team May 9, 2022
@cliAutomation cliAutomation added the external label May 9, 2022
@cliAutomation cliAutomation added this to Needs review 🤔 in The GitHub CLI May 9, 2022
@joshspicer joshspicer changed the title Joshspicer/add devcontainer add devcontainer May 9, 2022
@samcoe samcoe self-assigned this May 9, 2022
mislav
mislav approved these changes May 9, 2022
Copy link
Member

@mislav mislav left a comment

This looks great! Thank you.

And if this repository didn't have any .devcontainer configuration and someone opened a Codespace for it, wouldn't they also get a VS Code instance with Go preinstalled and the golang.go extension enabled? At least, that's how I remember hacking on CLI from Codespaces before. Does this configuration just enable us to have greater control over the Go version used?

.devcontainer/Dockerfile Outdated Show resolved Hide resolved
.devcontainer/Dockerfile Outdated Show resolved Hide resolved
.devcontainer/devcontainer.json Outdated Show resolved Hide resolved
The GitHub CLI automation moved this from Needs review 🤔 to Needs to be merged 🎉 May 9, 2022
@joshspicer
Copy link
Contributor Author

joshspicer commented May 9, 2022

In Codespaces, if a devcontainer isn't specified, you're given our default "kitchensink" image (defined here) which has a bunch of tools and runtimes installed (go included).

We're now defaulting to 1.18 in the default image, but that may change - so it is nice we now have the devcontainer to enforce consistency :)

@mislav mislav enabled auto-merge (squash) May 10, 2022
@mislav mislav merged commit bd1bf52 into cli:trunk May 10, 2022
5 checks passed
The GitHub CLI automation moved this from Needs to be merged 🎉 to Pending Release 🥚 May 10, 2022
@joshspicer joshspicer deleted the joshspicer/add-devcontainer branch May 10, 2022
@github-actions github-actions bot moved this from Pending Release 🥚 to Done 💤 in The GitHub CLI May 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external
Projects
The GitHub CLI
  
Done 💤
Development

Successfully merging this pull request may close these issues.

4 participants