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

🔨 make BaseScript more generic/powerful #8

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Conversation

qd-qd
Copy link
Member

@qd-qd qd-qd commented Nov 30, 2023

This light refactoring of BaseScript makes it more generic and powerful. Based on the flags you provide when running the script, it will either start the hardware wallet flow or use the private key you provide.

In addition, I added new comments for each script to detail how to use them.

Closes #3

This light refactoring of BaseScript makes it more generic and powerful.
Based on the flag you provide when running the script, it will either
start the hardware wallet flow or use the private key you provide.

In addition, I added new comments for each script to detail how to use
them.
@qd-qd qd-qd requested a review from lambertkevin November 30, 2023 14:19
@qd-qd qd-qd self-assigned this Nov 30, 2023
@qd-qd qd-qd changed the title 🔨 make BaseScript more generic/powerful 🔨 make BaseScript more generic/powerful Nov 30, 2023
Copy link

LCOV of commit 4dc0d76 during Tests #15

Summary coverage rate:
  lines......: 100.0% (21 of 21 lines)
  functions..: 100.0% (5 of 5 functions)
  branches...: no data found

Files changed coverage rate: n/a

@qd-qd qd-qd merged commit 4dc0d76 into main Jan 10, 2024
4 checks passed
@qd-qd qd-qd deleted the improve/base-script branch January 10, 2024 10:41
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.

🐛 Make the BaseScript contract greats again
1 participant