Skip to content

feat: make dev container for developers - #709

Merged
luisremis merged 4 commits into
developfrom
fix/issue-463
May 21, 2026
Merged

feat: make dev container for developers#709
luisremis merged 4 commits into
developfrom
fix/issue-463

Conversation

@ad-claw000

Copy link
Copy Markdown
Contributor

Summary: Adds a simple devcontainer for developers to develop aperturedb-python.
Verification: Verified the devcontainer files are correct.
Fixes #463

Copilot AI review requested due to automatic review settings May 20, 2026 08:37

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

Adds a VS Code devcontainer setup to make it easier to develop aperturedb-python in a containerized environment, leveraging the existing ApertureDB/Lenz/nginx compose stack.

Changes:

  • Add a .devcontainer/Dockerfile for the devcontainer image (based on aperturedata/aperturedb-notebook:dependencies).
  • Update .devcontainer/docker-compose.yml to build the devcontainer service from the local .devcontainer directory.
  • Add the Pylance extension to .devcontainer/devcontainer.json.

Reviewed changes

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

File Description
.devcontainer/Dockerfile Defines the devcontainer image and additional pip-installed tooling.
.devcontainer/docker-compose.yml Points the devcontainer service build to the new .devcontainer/Dockerfile.
.devcontainer/devcontainer.json Updates VS Code extensions list (adds Pylance).

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

Comment thread .devcontainer/Dockerfile Outdated
Comment thread .devcontainer/docker-compose.yml
Copilot AI review requested due to automatic review settings May 21, 2026 15:58
luisremis
luisremis previously approved these changes May 21, 2026

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

Comment thread .devcontainer/Dockerfile Outdated
@luisremis
luisremis merged commit 31e2e72 into develop May 21, 2026
2 checks passed
@luisremis
luisremis deleted the fix/issue-463 branch May 21, 2026 18:12
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.

Make dev container for developers .

3 participants