Skip to content

Conversation

@graydon
Copy link
Contributor

@graydon graydon commented Jan 30, 2026

Since there was a lot more feedback than I anticipated on #5111 I've split out the most important step of it for now: fixing the devcontainer to actually work with noble / support our current build. The devcontainer literally doesn't work currently, so this is fairly high value to land asap.

Copilot AI review requested due to automatic review settings January 30, 2026 02:54
leighmcculloch
leighmcculloch previously approved these changes Jan 30, 2026
Copy link

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

This PR updates the devcontainer configuration to work with Ubuntu Noble (24.04), addressing a critical issue where the devcontainer is currently non-functional. The changes update package sources paths, improve user creation logic, consolidate package installations, and modernize the VSCode configuration structure.

Changes:

  • Updated devcontainer from Ubuntu 20.04 to Ubuntu 24.04 (Noble)
  • Modernized VSCode devcontainer.json structure using the customizations field
  • Added development tool extensions for C++, Rust, and debugging support
  • Improved Dockerfile user/group creation logic to handle existing UIDs/GIDs
  • Updated apt sources path for Noble and consolidated package installations

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
.devcontainer/devcontainer.json Updated to Ubuntu 24.04 reference, modernized structure with customizations field, added VSCode extensions for development
.devcontainer/Dockerfile Updated apt sources path for Noble, improved user creation logic, consolidated package installation with gcc-14 and clang-20

@graydon graydon force-pushed the fix-devcontainer branch 3 times, most recently from 9aefbe3 to 8e638b1 Compare January 30, 2026 03:08
Copy link
Member

@leighmcculloch leighmcculloch left a comment

Choose a reason for hiding this comment

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

🚀

@graydon graydon enabled auto-merge January 30, 2026 04:02
@graydon graydon added this pull request to the merge queue Jan 30, 2026
Merged via the queue into stellar:master with commit db5c22d Jan 30, 2026
40 of 44 checks passed
@graydon graydon deleted the fix-devcontainer branch January 30, 2026 05:50
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.

2 participants