Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update API_LEVEL_21 #778

Merged
merged 14 commits into from
Sep 25, 2024
Merged

Conversation

abonnaudet-ledger
Copy link
Member

Description

Please provide a detailed description of what was done in this PR.
(And mention any links to an issue docs)

Changes include

  • Bugfix (non-breaking change that solves an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)
  • Tests
  • Documentation
  • Other (for changes that might not fit in any category)

Breaking changes

Please complete this section if any breaking changes have been made, otherwise delete it.

Additional comments

Please post additional comments in this section if you have them, otherwise delete it.

Stephane Portron and others added 12 commits September 24, 2024 13:36
(cherry picked from commit 16bcd2c)
(cherry picked from commit 3f2cb14)
(cherry picked from commit 3fbe861)
(cherry picked from commit 0b446a4)
(cherry picked from commit ed7420d)
(cherry picked from commit a940eb3)
Rename defines for experimental display features on Stax.

- `HAVE_DISPLAY_FAST_MODE` -> `HAVE_STAX_DISPLAY_FAST_MODE`
- `HAVE_CONFIGURABLE_DISPLAY_FAST_MODE` -> `HAVE_STAX_CONFIG_DISPLAY_FAST_MODE`

(cherry picked from commit f7d7ea6)
@codecov-commenter
Copy link

codecov-commenter commented Sep 24, 2024

Codecov Report

Attention: Patch coverage is 44.44444% with 5 lines in your changes missing coverage. Please review.

Project coverage is 35.67%. Comparing base (603cf09) to head (cbf19c3).
Report is 53 commits behind head on API_LEVEL_21.

Files with missing lines Patch % Lines
lib_ux_sync/src/ux_sync.c 57.14% 3 Missing ⚠️
lib_nbgl/src/nbgl_screen.c 0.00% 2 Missing ⚠️
Additional details and impacted files
@@                Coverage Diff                @@
##           API_LEVEL_21     #778       +/-   ##
=================================================
- Coverage         60.19%   35.67%   -24.53%     
=================================================
  Files                12       13        +1     
  Lines              1726     1595      -131     
=================================================
- Hits               1039      569      -470     
- Misses              687     1026      +339     
Flag Coverage Δ
unittests 35.67% <44.44%> (-24.53%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

kevin.meudic added 2 commits September 24, 2024 22:44
(cherry picked from commit ba52e4b)
@abonnaudet-ledger abonnaudet-ledger merged commit e908c59 into API_LEVEL_21 Sep 25, 2024
133 checks passed
@abonnaudet-ledger abonnaudet-ledger deleted the abo_update_api_level_21_2409 branch September 25, 2024 06:48
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