Skip to content

Conversation

@finger563
Copy link
Contributor

@finger563 finger563 commented Sep 30, 2025

Description

  • Add the new esp-cpp/esp-idf-size-delta action to the build step
  • Refactor build.yml to have cleaner comments about what to update
  • Add idf_size_release.yml for releases

Motivation and Context

Helps provide projects an easy, already-configured way to track how code changes are affecting their flash and ram usage between PRs and releases.

How has this been tested?

  • Checking that the build step still works on this PR
  • Checking that the summary shows properly in the action
  • Checking that the summary shows in a comment on this PR (and is updated by each run appropriately)

Screenshots (if appropriate, e.g. schematic, board, console logs, lab pictures):

CleanShot 2025-09-30 at 16 58 27 CleanShot 2025-09-30 at 16 58 13

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

Software

  • I have added tests to cover my changes.
  • I have updated the .github/workflows/build.yml file to add my new test to the automated cloud build github action.
  • All new and existing tests passed.
  • My code follows the code style of this project.

@finger563 finger563 self-assigned this Sep 30, 2025
@finger563 finger563 requested a review from Copilot September 30, 2025 18:15
Copy link

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 CI workflows for tracking ESP-IDF binary size changes using a new GitHub action. The implementation provides automated size delta tracking for both pull requests and releases to help monitor application size changes over time.

  • Added workflow for PR-based size tracking with automated comments
  • Added workflow for release-based size tracking with manual trigger option
  • Configured workflows for ESP-IDF v5.5 with customizable project parameters

Reviewed Changes

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

File Description
.github/workflows/idf_size_pr.yml PR workflow for automated size delta tracking on pull requests
.github/workflows/idf_size_release.yml Release workflow with tag comparison and manual dispatch triggers

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 added enhancement New feature or request ci labels Sep 30, 2025
@github-actions
Copy link

github-actions bot commented Sep 30, 2025

ESP-IDF Size Report for 'Template'

Metric Base PR Delta
FLASH 361,362 bytes (24.09%) 361,362 bytes (24.09%) 0 bytes (0.00%)
DRAM 15,672 bytes (8.67%) 15,672 bytes (8.67%) 0 bytes (0.00%)
IRAM 53,207 bytes (40.59%) 53,207 bytes (40.59%) 0 bytes (0.00%)
RAM (DRAM+IRAM) 68,879 bytes 68,879 bytes 0 bytes (0.00%)

FLASH uses app .bin size or json2 flash sum. RAM sums DRAM+IRAM via idf_size. Percentages shown when totals are available.
This report was generated by esp-idf-size-delta.

@finger563 finger563 requested a review from Copilot October 1, 2025 14:04
Copy link

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

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


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@finger563 finger563 merged commit acd43e3 into main Oct 1, 2025
5 of 6 checks passed
@finger563 finger563 deleted the feat/idf-size-ci branch October 1, 2025 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants