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

BC2A-1408: Fix duplicate symbols #9

Merged
merged 9 commits into from
Nov 20, 2023

Conversation

cedelavergne-ledger
Copy link

@cedelavergne-ledger cedelavergne-ledger commented Oct 31, 2023

Cleanup glyphs:

  • Removing duplicate gif files from sdk
  • Remove pre-generated glyphs.[ch] files from src/ directory
  • Add stubbed glyphs.h in unit-tests/ to fix compilation issue

This PR also updates the version

Cleanup the workflows and fix the warnings
Adding the guideline_enforcer workflow. It fails due to:

  • scan-build fails on the latest NanoS SDK, because of macro redefinition: UNUSED
  • scan-build fails on the latest NanoSP and NanoX, because of DEPRECATED crypto functions
  • Icons are not aligned with requirements (alpha channel found)
    Corrections will be done in another ticket...

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch 6 times, most recently from d36bc52 to afbe845 Compare November 3, 2023 15:02
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch 10 times, most recently from 24dd4b3 to d9287e1 Compare November 3, 2023 16:51
@codecov-commenter
Copy link

codecov-commenter commented Nov 3, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (b577bb9) 94.13% compared to head (41ac2c1) 94.14%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop       #9   +/-   ##
========================================
  Coverage    94.13%   94.14%           
========================================
  Files           10       10           
  Lines          716      717    +1     
========================================
+ Hits           674      675    +1     
  Misses          42       42           
Flag Coverage Δ
unittests 94.14% <ø> (+<0.01%) ⬆️

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.

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch 5 times, most recently from d89fd3d to bacb3a1 Compare November 6, 2023 08:42
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch from bacb3a1 to e61e04f Compare November 6, 2023 08:51
@sgliner-ledger
Copy link

@cedelavergne-ledger Sorry to ask you to do it but I think it would be good to add the guideline enforcer ? Even if it fails ? 🤔

@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch from 02ccc59 to 7be2ee0 Compare November 13, 2023 14:49
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch from 7be2ee0 to e324663 Compare November 13, 2023 15:16
@cedelavergne-ledger
Copy link
Author

@cedelavergne-ledger Sorry to ask you to do it but I think it would be good to add the guideline enforcer ? Even if it fails ? 🤔

Done. Some errors will have to be handled separately (mainly DEPRECATED functions)

sgliner-ledger
sgliner-ledger previously approved these changes Nov 13, 2023
@cedelavergne-ledger cedelavergne-ledger merged commit 6a5c835 into develop Nov 20, 2023
17 of 20 checks passed
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