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

Switch tests from zemu to ragger #67

Closed
wants to merge 4 commits into from

Conversation

tdejoigny-ledger
Copy link
Contributor

@tdejoigny-ledger tdejoigny-ledger commented Aug 7, 2023

First PR to initiate the transition from Zemu framework to Ragger framework.

As soon as this PR is merged, we can :

  • remove the Ethereum client from the repo
  • update the test requirements file "requirement.txt" to use directly the eth client available on pypi
  • update the swap test (and the CI) in order to build a known tx payload (instead of using a raw transaction from Etherscan) and to sign it (allowing the removal of flag "BYPASS_SIGNATURES=1" in ETH app)
  • use the correct branch for plugin-sdk sub-module (>>develop branch)

@tdejoigny-ledger tdejoigny-ledger force-pushed the tdj/switch_from_zemu_to_ragger branch 2 times, most recently from 0e1f279 to 6aebdfd Compare September 12, 2023 07:49
@tdejoigny-ledger tdejoigny-ledger marked this pull request as ready for review September 12, 2023 08:02
@tdejoigny-ledger tdejoigny-ledger changed the title Tdj/switch from zemu to ragger Switch tests from zemu to ragger Sep 12, 2023
.gitignore Outdated Show resolved Hide resolved
.gitignore Outdated Show resolved Hide resolved
@tdejoigny-ledger tdejoigny-ledger force-pushed the tdj/switch_from_zemu_to_ragger branch 4 times, most recently from 8cd8c5b to b434bb7 Compare October 26, 2023 16:13
@fbeutin-ledger
Copy link
Contributor

#105

@apaillier-ledger apaillier-ledger deleted the tdj/switch_from_zemu_to_ragger branch February 29, 2024 09:25
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.

None yet

2 participants