Skip to content

Conversation

@Pabl0cks
Copy link
Collaborator

This PR adds documentation for yarn account:reveal-pk command for Hardhat and Foundry.

Got added to SE-2 in scaffold-eth/scaffold-eth-2#1094 and scaffold-eth/scaffold-eth-2#1091

@Pabl0cks Pabl0cks requested a review from technophile-04 May 20, 2025 13:57
@vercel
Copy link

vercel bot commented May 20, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
scaffold-eth-2-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 21, 2025 7:56am


You will get prompted to paste your private key and set the encryption password. It will store your encrypted private key in your `.env` file.

To reveal the private key of your currently configured deployer account (from `.env`), you can run:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For me, it'd make sense to have reveal-pk as the last command

  • generate
  • account:import
  • account
  • reveal-pk

Not a big deal tho! But maybe we could move the account instructions inside each hardhat / foundry tab so we can put it before reveal-pk? (even if we are duplicating it)

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agree! It was actually my initial approach. but then I changed it. Makes sense to follow this workflow 🙏

I've added bold to the 4 actions to make sure they stand out (using subtitles felt a bit too much but could be also an option!)

Copy link
Collaborator

@technophile-04 technophile-04 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tysm @Pabl0cks!! Just added a small comment feel free to ignore 🙌

@technophile-04 technophile-04 merged commit e52cef8 into main May 21, 2025
3 checks passed
@technophile-04 technophile-04 deleted the reveal-pk branch May 21, 2025 09:17
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.

4 participants