Skip to content

Conversation

@CarbonNeuron
Copy link
Contributor

Description

This pull request updates the .vscode/settings.json file to improve code formatting consistency and developer experience. The main change is the introduction of automatic code formatting using clang-format, which will help ensure a uniform code style across the project.

Code formatting improvements:

  • Enabled clang-format as the default formatter by setting "C_Cpp.formatting": "clangFormat" and specifying the .clang-format configuration file for formatting rules ("C_Cpp.clang_format_style": "file:/workspaces/espp/.clang-format").
  • Enabled automatic formatting on save for modified lines only by setting "editor.formatOnSave": true and "editor.formatOnSaveMode": "modifications".Added formatting settings for C++ and enabled format on save.

Motivation and Context

How has this been tested?

Ran in own container

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

Added formatting settings for C++ and enabled format on save.
Copilot AI review requested due to automatic review settings December 19, 2025 22:21
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 configures automatic code formatting in VS Code using clang-format to ensure consistent code style across the project. The settings enable format-on-save for modified lines only, using the project's existing .clang-format configuration file.

Key Changes:

  • Enabled clang-format as the default C/C++ formatter with reference to project's .clang-format file
  • Configured automatic formatting on save for modified lines only
  • Added missing comma after existing idf.flashType setting

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

@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 merged commit e94ee66 into esp-cpp:main Dec 19, 2025
1 check 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