Skip to content

Conversation

@ohhmm
Copy link
Contributor

@ohhmmohhmm commented Dec 23, 2023

Make "Reopen in Container" command work in Visual Studio code.

bnb
bnb approved these changes Dec 26, 2023
Copy link
Contributor

@bnbbnb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be correct, based off this documentation. My understanding is that there's not a really great way to test something like this without merging - happy to be told I'm wrong, but this LGTM as the person who originally built the devcontainer.

If it's not, I'm happy to take responsibility for fixing it 👍🏻

Copy link
Member

@anonriganonrig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@anonriganonrig added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Dec 26, 2023
@aduh95aduh95 added the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 31, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Dec 31, 2023
@nodejs-github-botnodejs-github-bot merged commit 5fb6305 into nodejs:mainDec 31, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 5fb6305

@ohhmmohhmm deleted the dockerfix branch December 31, 2023 11:18
RafaelGSS pushed a commit that referenced this pull request Jan 2, 2024
PR-URL: #51271 Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@RafaelGSSRafaelGSS mentioned this pull request Jan 2, 2024
richardlau pushed a commit that referenced this pull request Mar 25, 2024
PR-URL: #51271 Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
richardlau pushed a commit that referenced this pull request Mar 25, 2024
PR-URL: #51271 Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Yagiz Nizipli <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
@richardlaurichardlau mentioned this pull request Mar 25, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@ohhmm@nodejs-github-bot@bnb@lpinca@anonrig@aduh95