Skip to content

Conversation

@CarbonNeuron
Copy link
Contributor

Adds .vscode and .devcontainer configs for automatically setting up development environments.

Motivation and Context

Makes it easier to work on project

How has this been tested?

I ran "Reopen in dev container" in vscode, and it built, and executed the container correctly.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation Update
  • Hardware (schematic, board, system design) change
  • Software change

Checklist:

  • My change requires a change to the documentation.
  • I have added / updated the documentation related to this change via either README or WIKI

Copilot AI review requested due to automatic review settings December 18, 2025 17:02
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

This PR adds VS Code and DevContainer configuration files to streamline development environment setup for the ESP-IDF project. The configurations enable automatic container-based development with pre-configured tooling and extensions.

Key changes:

  • Added VS Code workspace settings for ESP-IDF IntelliSense, Python environment, and device configuration
  • Created DevContainer configuration with ESP-IDF v5.5.1 Docker image and required VS Code extensions
  • Configured C/C++ IntelliSense properties for RISC-V toolchain and ESP-IDF component paths

Reviewed changes

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

Show a summary per file
File Description
.vscode/settings.json Configures ESP-IDF extension with Python path, serial port, and flash settings
.vscode/launch.json Defines debug configurations for GDB and ESP-IDF debugging
.vscode/c_cpp_properties.json Sets up C/C++ IntelliSense with RISC-V compiler and ESP-IDF include paths
.devcontainer/devcontainer.json Configures DevContainer with ESP-IDF paths, extensions, and privileged mode
.devcontainer/Dockerfile Creates container based on ESP-IDF v5.5.1 with locale settings and auto-sourcing

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

ENV LC_ALL=C.UTF-8
ENV LANG=C.UTF-8

RUN apt-get update -y && apt-get install udev -y
Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

The apt-get command should include '&& apt-get clean && rm -rf /var/lib/apt/lists/*' at the end to reduce the Docker image size by removing package cache files.

Suggested change
RUN apt-get update -y && apt-get install udev -y
RUN apt-get update -y && apt-get install udev -y && apt-get clean && rm -rf /var/lib/apt/lists/*

Copilot uses AI. Check for mistakes.
@CarbonNeuron
Copy link
Contributor Author

Probably will later need to configure clang, but this is a good starting point

@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 merged commit e19da2c into esp-cpp:main Dec 18, 2025
4 of 5 checks passed
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