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

fix: missing arg in zk init #1805

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thomas-nguy
Copy link

What ❔

Make the skip-test-token-deployment option available

Why ❔

We don't want to deploy test tokens when deploying on Sepolia or other networks
(current test erc20 deployment consumes a lot of gas)

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via zk fmt and zk lint.
  • Spellcheck has been run via zk spellcheck.
  • Linkcheck has been run via zk linkcheck.

@thomas-nguy thomas-nguy changed the title fix (script) : missing arg in zk init fix : missing arg in zk init Apr 26, 2024
@thomas-nguy thomas-nguy changed the title fix : missing arg in zk init fix: missing arg in zk init Apr 26, 2024
@thomas-nguy thomas-nguy force-pushed the thomas/fix-missing-arg-in-init branch from c656940 to 3043f8e Compare April 26, 2024 09:26
@thomas-nguy thomas-nguy force-pushed the thomas/fix-missing-arg-in-init branch from 3043f8e to 3bece07 Compare May 8, 2024 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant