Skip to content

Actually, we want mongosh...#36

Merged
ikogan merged 1 commit into
mainfrom
cs/actually-mongosh
Jun 8, 2026
Merged

Actually, we want mongosh...#36
ikogan merged 1 commit into
mainfrom
cs/actually-mongosh

Conversation

@ikogan

@ikogan ikogan commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings June 8, 2026 19:16
@ikogan
ikogan merged commit 216ce12 into main Jun 8, 2026
3 of 4 checks passed

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 devcontainer base image to install MongoDB Shell (mongosh) instead of the legacy MongoDB CLI (mongocli), aligning the container tooling with the desired MongoDB client interface.

Changes:

  • Replaces the MONGOCLI_VERSION build arg with MONGOSH_VERSION.
  • Updates the Dockerfile install steps to download and install mongosh from the mongodb-js/mongosh GitHub releases.
  • Simplifies architecture handling to use dpkg --print-architecture directly for the mongosh artifact name.

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

Comment thread Dockerfile
mkdir -p /tmp/ohio-pkgs && \
curl -Lo /tmp/ohio-pkgs/mongocli.deb https://github.com/mongodb/mongodb-cli/releases/download/mongocli%2Fv"${MONGOCLI_VERSION}"/mongocli_"${MONGOCLI_VERSION}"_linux_"${MONGOCLI_ARCHITECTURE}".deb && \
apt-get install -y /tmp/ohio-pkgs/mongocli.deb && \
curl -Lo /tmp/ohio-pkgs/mongosh.deb https://github.com/mongodb-js/mongosh/releases/download/v"${MONGOSH_VERSION}"/mongodb-mongosh_"${MONGOSH_VERSION}"_"${MONGOSH_ARCHITECTURE}".deb && \
@ikogan
ikogan deleted the cs/actually-mongosh branch June 22, 2026 19:33
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