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

chore: add vscode bazel container #165

Closed
wants to merge 1 commit into from

Conversation

Aghassi
Copy link
Contributor

@Aghassi Aghassi commented Jun 18, 2022

This change adds the necessary files so that if contributing via
vscodespaces you have bazel on your path via bazelisk and don't
need to do extra work to set it up

I also created an upstream issue for fixes I added to the container
to keep it ergonomic:

microsoft/vscode-dev-containers#1507
microsoft/vscode-dev-containers#1508

This change adds the necessary files so that if contributing via
vscodespaces you have bazel on your path via bazelisk and don't
need to do extra work to set it up
@alexeagle
Copy link
Collaborator

Hm, this seems really cool to have things 'just work' when editing on GH, but this is a lot of SLOC for us to maintain that none of us understands

@Aghassi
Copy link
Contributor Author

Aghassi commented Jul 17, 2022

Yeah unfortunately that's how VSCodespaces bootstraps. Not sure if there is a cleaner way... it's basically what Microsoft says to do at this point 😭

@alexeagle
Copy link
Collaborator

Reading recent comments on your upstream issues, I get the sense it's not time for us to adopt this yet. @Aghassi as you know I'm super-keen on the idea though. If we could get this working in a more ergonomic way, I'd love to have every one of our OSS Bazel-related repos contain the codespaces solution.

@alexeagle alexeagle closed this Oct 13, 2022
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.

3 participants