Skip to content

Use ubuntu base image for devcontainer#514

Merged
ludeeus merged 4 commits intomainfrom
change-devcontainer
Feb 27, 2026
Merged

Use ubuntu base image for devcontainer#514
ludeeus merged 4 commits intomainfrom
change-devcontainer

Conversation

@ludeeus
Copy link
Copy Markdown
Owner

@ludeeus ludeeus commented Feb 27, 2026

Description

SSIA.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (which adds functionality)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements

Checklist

  • Tests pass locally (scripts/test)
  • Code has been formatted (scripts/lint)
  • Tests added for new functionality (if applicable)

Copilot AI review requested due to automatic review settings February 27, 2026 11:35
Copy link
Copy Markdown
Contributor

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

Updates the devcontainer to use an Ubuntu base image and adjusts the setup flow to provision Python via uv, aligning the development environment with the repository’s uv-based workflow.

Changes:

  • Switch devcontainer image from the Python devcontainer image to base:ubuntu.
  • Add a Python installation step (uv python install) to scripts/setup before bootstrapping dependencies.

Reviewed changes

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

File Description
scripts/setup Installs Python via uv before running scripts/bootstrap.
.devcontainer.json Changes the devcontainer base image to Ubuntu.

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

Copilot AI review requested due to automatic review settings February 27, 2026 11:39
Copy link
Copy Markdown
Contributor

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

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


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

@ludeeus ludeeus merged commit 59c80d6 into main Feb 27, 2026
7 checks passed
@ludeeus ludeeus deleted the change-devcontainer branch February 27, 2026 11:42
@ludeeus ludeeus added the maintenance Maintenance and refactoring label Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance and refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants