Skip to content

Conversation

@sam1el
Copy link

@sam1el sam1el commented Jun 16, 2025

Updates to development container configuration:

  • Changed the source of the mocha feature from ghcr.io/devcontainers-contrib to ghcr.io/devcontainers-extra for better support and maintenance.
  • Updated the java feature to specify mavenVersion as 3.9.10 for more precise Maven version control.
  • Upgraded the python feature version from 3.8 to 3.11 to use a more recent Python release.

This addresses current issues causing the codespace to be broken when loading. Changes will load the codespace now without errors.

@sam1el sam1el marked this pull request as ready for review June 16, 2025 16:37
@davihern davihern requested a review from Copilot June 17, 2025 18:39
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 development container configuration to improve feature maintenance and use more recent runtimes.

  • Switched the mocha feature source from devcontainers-contrib to devcontainers-extra
  • Added a mavenVersion pin for the Java feature alongside installMaven
  • Upgraded the Python feature from 3.8 to 3.11

},
"ghcr.io/devcontainers-contrib/features/mocha:2": {
"ghcr.io/devcontainers-extra/features/mocha:2": {
"version": "latest"
Copy link

Copilot AI Jun 17, 2025

Choose a reason for hiding this comment

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

[nitpick] Consider pinning the Mocha feature version instead of using latest to ensure reproducible container builds.

Suggested change
"version": "latest"
"version": "2.0.0"

Copilot uses AI. Check for mistakes.
},
"ghcr.io/devcontainers/features/python:1": {
"version": "3.8"
"version": "3.11"
Copy link

Copilot AI Jun 17, 2025

Choose a reason for hiding this comment

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

[nitpick] To avoid unexpected changes when Python 3.11.x patches are released, consider specifying the full patch version (e.g., 3.11.2).

Suggested change
"version": "3.11"
"version": "3.11.2"

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

@davihern davihern left a comment

Choose a reason for hiding this comment

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

lgtm

@davihern davihern merged commit 6f66712 into microsoft:main Jun 17, 2025
6 checks passed
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