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

lib: starkli as lib #102

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

ICavlek
Copy link

@ICavlek ICavlek commented Nov 6, 2024

With this PR, updates on starkli in general and couple of features are introduced. Inside beerus, it is being used as part of the test case which simulates whole starknet environment and uses all of it's major tools: starkli, katana, scarb and beerus. List of the commits this PR consists of:

  • starkli as lib - possibility to include starkli as a library
  • flag to persist logger - multiple initializations of logger are not allowed, therefore possibility to skip it when calling multiple functions from starkli as library
  • flag for skipping prompt in account deploy - it has been marked as TODO and it won't block the test case
  • call subfunction - extracted functionality from call to a separate function which returns Vec<Felt> that can be verified in test case

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.

1 participant