Log the OpenThread version when the instance is initialized - #5
Merged
Conversation
Also a small, deliberate flash size increase to exercise the size report on pull requests. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Size reportFirmware size at a01f552 compared to the base branch at 09af6f1. Increases above 0.2%: None. Regions
Sections
|
…tion Each MCU leg now emits a JSON measurement; a final job renders all of them as one report with the increases above the threshold, all regions and all sections, so that a change can be compared across chips. Unchanged sections are listed too. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
ivmarkov
added a commit
that referenced
this pull request
Sep 8, 2026
* Implement a size-check * Log the OpenThread version when the instance is initialized (#5) * Log the OpenThread version when the instance is initialized Also a small, deliberate flash size increase to exercise the size report on pull requests. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Size report: one report across all MCUs, listing every region and section Each MCU leg now emits a JSON measurement; a final job renders all of them as one report with the increases above the threshold, all regions and all sections, so that a change can be compared across chips. Unchanged sections are listed too. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Size report: measure all examples, not only basic_udp Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> * Switch to the released elfsize * Rename cargo elfsize to just elfsize * Address code review feedback --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Also a small, deliberate flash size increase to exercise the size report on pull requests.