Skip to content

fix: update devcontainer's docker image to match github#1323

Closed
vegerot wants to merge 1 commit into
facebook:mainfrom
vegerot:pr1323
Closed

fix: update devcontainer's docker image to match github#1323
vegerot wants to merge 1 commit into
facebook:mainfrom
vegerot:pr1323

Conversation

@vegerot
Copy link
Copy Markdown
Contributor

@vegerot vegerot commented May 27, 2026

No description provided.

Copilot AI review requested due to automatic review settings May 27, 2026 20:30
@meta-cla meta-cla Bot added the CLA Signed label May 27, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the VS Code devcontainer configuration to build from a different Dockerfile (manylinux 2.34), likely to align the local dev environment with a new base image/toolchain.

Changes:

  • Switched the devcontainer build Dockerfile from sapling-cli-ubuntu-22.04.Dockerfile to manylinux_2_34.Dockerfile.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -2,7 +2,7 @@
"name": "Sapling",
"build": {
"context": "..",
@vegerot vegerot closed this May 27, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 27, 2026

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D106554635. (Because this pull request was imported automatically, there will not be any future comments.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants