Feature update to platformio #15
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the build system from Arduino CLI to PlatformIO for both ESP8266 and Arduino firmware projects. The changes enable better dependency management, consistent build environments, and improved CI/CD workflows.
- Replaced Arduino CLI build system with PlatformIO configuration
- Added new PlatformIO configuration files for both firmware projects
- Updated GitHub Actions workflow to use PlatformIO instead of Arduino CLI
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| Firmware_ESP8266/platformio.ini | Updated PlatformIO configuration with streamlined settings and glob patterns |
| Firmware_ESP8266/.gitignore | Added PlatformIO and VS Code build artifacts to ignore list |
| Firmware_Arduino/platformio.ini | New PlatformIO configuration for Arduino Pro Mini 8MHz build |
| Firmware_Arduino/.gitignore | Added PlatformIO and VS Code build artifacts to ignore list |
| .github/workflows/build.yaml | Completely rewritten CI workflow to use PlatformIO with matrix builds |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| [platformio] | ||
| default_envs = esp8266 | ||
| src_dir = Firmware_ESP8266 | ||
| src_dir = . |
There was a problem hiding this comment.
Setting src_dir to '.' may cause confusion as it changes the default PlatformIO source directory structure. Consider using the standard 'src' directory or documenting why this deviation is necessary.
| src_dir = . |
| -<*.c> | ||
| +<**/*.ino> | ||
| +<**/*.cpp> | ||
| -<**/*.c> |
There was a problem hiding this comment.
The build_src_filter excludes all .c files with '-<**/*.c>'. This could prevent compilation of necessary C source files if any exist in the project. Consider being more specific about which C files to exclude or remove this filter if not needed.
| -<**/*.c> |
| src_dir = . | ||
|
|
There was a problem hiding this comment.
Setting src_dir to '.' may cause confusion as it changes the default PlatformIO source directory structure. Consider using the standard 'src' directory or documenting why this deviation is necessary.
| src_dir = . |
| -<**/*.c> | ||
|
|
There was a problem hiding this comment.
The build_src_filter excludes all .c files with '-<**/*.c>'. This could prevent compilation of necessary C source files if any exist in the project. Consider being more specific about which C files to exclude or remove this filter if not needed.
| -<**/*.c> |
…ControlPanel into feature_update_to_platformio
No description provided.