Skip to content

Fixes and improvements to devcontainer#43

Open
douglaswinter wants to merge 2 commits into
mainfrom
dw/devcontainer
Open

Fixes and improvements to devcontainer#43
douglaswinter wants to merge 2 commits into
mainfrom
dw/devcontainer

Conversation

@douglaswinter
Copy link
Copy Markdown
Collaborator

  1. The most important fix is to use node 22 for the dev container. The previous image was causing an error when we try to install pnpm@latest. This is the node version we are already using to create app images.
  2. Install less for e.g. git pagination
  3. Remove unnecessary extension

1) The most important fix is to use node 22 for the dev container. The
   previous image was causing an error when we try to install the latest
pnpm.
2) Install less for e.g. git pagination
3) Remove unnecessary extension.
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

Updates the repository’s devcontainer configuration to align with the Node version used for app image builds and to improve the developer experience inside the container.

Changes:

  • Bump the devcontainer base image from Node 20 to Node 22.
  • Install less in the devcontainer for improved CLI usability (e.g., pagination).
  • Remove the TypeScript “next” VS Code extension and adjust VS Code settings in devcontainer.json.

Reviewed changes

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

File Description
.devcontainer/Dockerfile Switches base image to Node 22 and adds less to installed packages.
.devcontainer/devcontainer.json Updates VS Code extensions list and modifies a TypeScript-related editor setting key.

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

Comment thread .devcontainer/devcontainer.json
Comment thread .devcontainer/Dockerfile Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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