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: Remove duplicate apdu_parser #1

Merged
merged 3 commits into from
Nov 20, 2023

Conversation

cedelavergne-ledger
Copy link

@cedelavergne-ledger cedelavergne-ledger commented Nov 17, 2023

Function apdu_parser was duplicated between the App itself and the SDK. This PR allows to clean up the code:

  • Remove the useless app code
  • Fix the header files include
  • Fix type redefinition
  • Fix the unit-tests accordingly

This PR also updates the version (and thus the corresponding tests results and screenshots)

- 1st definition in SDK
- 2nd definition in the Application (same filename also)
- Fix dependencies (header files, definitions)
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

lpascal-ledger
lpascal-ledger previously approved these changes Nov 17, 2023
@cedelavergne-ledger cedelavergne-ledger force-pushed the cev/B2CA-1408_conflict-name branch 2 times, most recently from 8ed0275 to 3c6d24f Compare November 17, 2023 15:47
@cedelavergne-ledger cedelavergne-ledger merged commit 6da0e28 into develop Nov 20, 2023
33 of 34 checks passed
lemois-1337 pushed a commit to lemois-1337/app-karlsen that referenced this pull request Oct 2, 2024
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.

4 participants