update(devcontainer): update claude and golang version to 1.25.9#2885
update(devcontainer): update claude and golang version to 1.25.9#2885clamoriniere wants to merge 3 commits intomainfrom
Conversation
To be able to use the last version of claud code, we need to use for now a customer feature release until this PR is merged anthropics/devcontainer-features#37
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 65aea2da77
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
| @@ -1,13 +1,14 @@ | |||
| { | |||
| "image": "mcr.microsoft.com/devcontainers/go:dev-1.25", | |||
| "image": "golang:1.25.9", | |||
There was a problem hiding this comment.
Switching the base image to golang:1.25.9 removes the default vscode user provided by the previous devcontainer image, so this container now runs as root unless a remoteUser is explicitly configured. In local Linux devcontainer setups, the postStartCommand (make install-tools) will create root-owned files in the bind-mounted repo (for example under bin/), which blocks normal host-side edits/cleanup and is a significant workflow regression. Please keep a devcontainer base that defines vscode or explicitly set up and use a non-root user.
Useful? React with 👍 / 👎.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2885 +/- ##
=======================================
Coverage 40.06% 40.06%
=======================================
Files 319 319
Lines 28039 28039
=======================================
Hits 11233 11233
Misses 15983 15983
Partials 823 823
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
3 participants
What does this PR do?
To be able to use the last version of claud code, we need to use for now a customer feature release until this PR is merged anthropics/devcontainer-features#37
Motivation
Additional Notes
Anything else we should know when reviewing?
Minimum Agent Versions
Are there minimum versions of the Datadog Agent and/or Cluster Agent required?
Describe your test plan
Write there any instructions and details you may have to test your PR.
Checklist
bug,enhancement,refactoring,documentation,tooling, and/ordependenciesqa/skip-qalabel