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

Foundry keystore #14

Merged
merged 6 commits into from
May 28, 2024
Merged

Foundry keystore #14

merged 6 commits into from
May 28, 2024

Conversation

uF4No
Copy link
Contributor

@uF4No uF4No commented May 23, 2024

What 💻

  • Removes .env files from Foundry templates.
  • Adds instructions to setup a Foundry keystore for PK management
  • Updates example scripts to use keystore

Why ✋

  • More secure!
  • We use these templates in the docs and we're updating them to use keystores by default

Evidence 📷

Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

Notes 📝

  • Any notes/thoughts that the reviewers should know prior to reviewing the code?

@uF4No uF4No requested a review from a team as a code owner May 23, 2024 10:19
Co-authored-by: Nicolas Villanueva <nicolasvillanueva@msn.com>
templates/quickstart/foundry/factory/README.md Outdated Show resolved Hide resolved
templates/quickstart/foundry/hello-zksync/README.md Outdated Show resolved Hide resolved
templates/quickstart/foundry/testing/README.md Outdated Show resolved Hide resolved
templates/quickstart/foundry/testing/README.md Outdated Show resolved Hide resolved
itsacoyote pushed a commit to matter-labs/zksync-docs that referenced this pull request May 28, 2024
# What 💻 
* Updates 101 getting started to setup a keystore
* Updates all 101 Foundry tutorials to use keystore in commands to
deploy/send txs
* Update all commands in Foundry guide with examples of keystore use and
PK.
* Additionally, fixes a partial not correctly displayed in paymaster
page

# Why ✋
* Fixes DEVRL676
* To be merged with
matter-labs/zksync-contract-templates#14

# Evidence 📷
Include screenshots, screen recordings, or `console` output here
demonstrating that your changes work as intended

<!-- All sections below are optional. You can uncomment any section
applicable to your Pull Request. -->

<!-- # Notes 📝
* Any notes/thoughts that the reviewers should know prior to reviewing
the code? -->

---------

Co-authored-by: Nicolas Villanueva <nicolasvillanueva@msn.com>
@itsacoyote itsacoyote merged commit 49f1476 into main May 28, 2024
3 checks passed
@itsacoyote itsacoyote deleted the feat/foundry-keystore branch May 28, 2024 21:52
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.

3 participants