Skip to content

chore: CI Build Summary - firmware stats, firmware artifact#601

Merged
daveallie merged 30 commits intocrosspoint-reader:masterfrom
osteotek:chore-pr-firmware-stats
Feb 5, 2026
Merged

chore: CI Build Summary - firmware stats, firmware artifact#601
daveallie merged 30 commits intocrosspoint-reader:masterfrom
osteotek:chore-pr-firmware-stats

Conversation

@osteotek
Copy link
Member

@osteotek osteotek commented Jan 28, 2026

Summary

  • adds to the Action Summary firmware RAM and Flash usage
  • splits cppcheck, clang-format and build into separate jobs, more apparent what had failed.
  • attaches the built artifact (firmware.bin) to the Action Summary for easier testing.
image

@osteotek osteotek changed the title feat: add firmware stats extraction and PR commenting chore: add firmware stats extraction and PR commenting Jan 28, 2026
@osteotek osteotek closed this Jan 28, 2026
@osteotek osteotek reopened this Jan 28, 2026
@osteotek osteotek requested a review from daveallie January 30, 2026 12:15
@osteotek

This comment was marked as outdated.

@osteotek osteotek changed the title chore: add firmware stats extraction and PR commenting chore: PR comment - firmware stats, firmware artifact Jan 30, 2026
Copilot AI review requested due to automatic review settings February 1, 2026 08:05

This comment was marked as outdated.

@daveallie daveallie force-pushed the chore-pr-firmware-stats branch from efa86ea to a540edd Compare February 1, 2026 08:40
@daveallie
Copy link
Member

Even with this setting it seems to through a fit

CleanShot 2026-02-01 at 20 07 08

@osteotek osteotek changed the title chore: PR comment - firmware stats, firmware artifact chore: CI Build Summary - firmware stats, firmware artifact Feb 1, 2026
@osteotek
Copy link
Member Author

osteotek commented Feb 1, 2026

@daveallie turns out creating comments from PRs that come from forks is way more complicated... Github (rightfully) doesn't trust such PRs with full set of permissions. Had to modify my proposed changes to create CI Build action summary with uploaded artifact and build RAM and Flash stats. It can be seen by pressing CI (build) action and going to Summary. Less obvious, but hopefully still useful

@daveallie daveallie merged commit 216dbc8 into crosspoint-reader:master Feb 5, 2026
3 checks passed
jdk2pq added a commit to jdk2pq/crosspoint-reader that referenced this pull request Feb 5, 2026
* master:
  feat: add shift lock to KeyboardEntryActivity (crosspoint-reader#513)
  feat: rename and move in file manager (crosspoint-reader#630)
  feat: Implement fix for sunlight fading issue (crosspoint-reader#603)
  chore: Add PR title check on sync (crosspoint-reader#698)
  feat: Go To Position for epubs (crosspoint-reader#666)
  feat: Calibre Web Automated (CWA) koreader sync server support (crosspoint-reader#594)
  chore: Add CI check job to consolidate status (crosspoint-reader#696)
  chore: CI Build Summary - firmware stats, firmware artifact (crosspoint-reader#601)
  feat: quick rotate option in epub reader menu (crosspoint-reader#685)
  feat(settings): add "Cover + Custom" sleep screen mode (crosspoint-reader#582)
  fix: Artifacts on Thumb on Home Screen (crosspoint-reader#662)
  feat: holding back button while booting, boots to home screen as a mean of escaping boot loop (crosspoint-reader#587)
  docs: Add small SCOPE.md and GOVERNANCE.md documents (crosspoint-reader#640)
  feat: front button remapper (crosspoint-reader#664)
  feat: UI themes, Lyra (crosspoint-reader#528)
  feat: Add CSS parsing and CSS support in EPUBs (crosspoint-reader#411)
  fix: move http upload state to heap (crosspoint-reader#657)
Unintendedsideeffects pushed a commit to Unintendedsideeffects/crosspoint-reader that referenced this pull request Feb 17, 2026
…nt-reader#601)

## Summary

* adds to the Action Summary firmware RAM and Flash usage
* splits cppcheck, clang-format and build into separate jobs, more
apparent what had failed.
* attaches the built artifact (firmware.bin) to the Action Summary for
easier testing.


<img width="853" height="456" alt="image"
src="https://github.com/user-attachments/assets/61d21c3c-58c2-42f3-8d8e-4cbedcf193e9"
/>

---------

Co-authored-by: Copilot <[email protected]>
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.

3 participants