Skip to content

MAINT set up GitHub Copilot in devcontainer#1431

Merged
romanlutz merged 12 commits intoAzure:mainfrom
romanlutz:romanlutz/devcontainer-gh-copilot-cli
Mar 4, 2026
Merged

MAINT set up GitHub Copilot in devcontainer#1431
romanlutz merged 12 commits intoAzure:mainfrom
romanlutz:romanlutz/devcontainer-gh-copilot-cli

Conversation

@romanlutz
Copy link
Contributor

@romanlutz romanlutz commented Mar 2, 2026

Sets up GitHub Copilot in the devcontainer:

- Installs the Copilot CLI (@github/copilot@0.0.421) globally via npm in the Dockerfile
- Adds the github.copilot and github.copilot-chat VS Code extensions to devcontainer.json

This makes both the Copilot CLI (for terminal use) and Copilot Chat (in the VS Code sidebar) available inside the devcontainer.

GitHub Copilot CLI appears to be more convenient in many situations. This change adds it into our devcontainer so that it's available inside.

This can be used quite easily with the following button in the GitHub Copilot dropdown in VS Code

image

and just renders inline

image

which can be split to the right just like the GitHub Copilot Chat view.

Copilot AI review requested due to automatic review settings March 2, 2026 15:00
Copy link
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

Adds GitHub Copilot tooling to the devcontainer so Copilot is available inside the containerized dev environment.

Changes:

  • Install @github/copilot globally in the devcontainer image.
  • Add github.copilot and github.copilot-chat to the devcontainer’s VS Code extension list.
  • Update frontend/package-lock.json (includes a Rollup version change).

Reviewed changes

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

File Description
frontend/package-lock.json Updates locked frontend dependencies (notably Rollup + platform-specific Rollup binaries).
.devcontainer/devcontainer.json Adds Copilot and Copilot Chat extensions to the container’s VS Code setup.
.devcontainer/Dockerfile Installs Copilot CLI via npm during devcontainer image build.
Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

Copilot AI review requested due to automatic review settings March 2, 2026 19:17
Copy link
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 3 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

romanlutz and others added 4 commits March 2, 2026 14:41
Reset the lockfile to main and re-ran npm install to eliminate
spurious metadata churn (libc fields, peer flag noise) that was
introduced by a different npm version. The lockfile now has a
minimal, consistent diff.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@romanlutz romanlutz force-pushed the romanlutz/devcontainer-gh-copilot-cli branch from 2118152 to 1138bda Compare March 2, 2026 22:41
Copilot AI review requested due to automatic review settings March 3, 2026 00:42
Copy link
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 3 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

Copilot AI review requested due to automatic review settings March 3, 2026 05:01
Copy link
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 3 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • frontend/package-lock.json: Language not supported

romanlutz and others added 2 commits March 2, 2026 21:10
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 3, 2026 05:29
Copy link
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 no new comments.

romanlutz and others added 2 commits March 3, 2026 22:32
Copilot AI review requested due to automatic review settings March 4, 2026 06:43
@romanlutz romanlutz changed the title MAINT set up GitHub Copilot CLI in devcontainer MAINT set up GitHub Copilot in devcontainer Mar 4, 2026
Copy link
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 1 comment.


You can also share your feedback on Copilot code review. Take the survey.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 4, 2026 17:25
Copy link
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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Contributor

@hannahwestra25 hannahwestra25 left a comment

Choose a reason for hiding this comment

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

yesss!

@hannahwestra25 hannahwestra25 self-assigned this Mar 4, 2026
@romanlutz romanlutz merged commit af13f75 into Azure:main Mar 4, 2026
41 of 42 checks passed
@romanlutz romanlutz deleted the romanlutz/devcontainer-gh-copilot-cli branch March 4, 2026 21:00
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.

3 participants