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

Consider better logging throughout the project #125

Open
linear bot opened this issue Jan 18, 2024 · 1 comment
Open

Consider better logging throughout the project #125

linear bot opened this issue Jan 18, 2024 · 1 comment

Comments

@linear
Copy link

linear bot commented Jan 18, 2024

When deploying the cap table using forge script, critical information like this is currently not highlighted enough. It degrades developer experience (which is important for collaboration). If only a few people know what to do and where to look, it'll be harder to onboard new devs

forge script script/CapTableFactory.s.sol --fork-url http://*********:8449 --broadcast
[⠒] Compiling...
No files changed, compilation skipped
Script ran successfully.

== Logs ==
  Upgrading CapTableFactory with CapTable implementation
  Deploying CapTableFactory and CapTable implementation
  CapTable implementation deployed at: 0x888afDfCf11E545D5E004B6Fb66008d9787c9579
  CapTableFactory deployed at: 0x745258b51eB8CAa675DD041bAFC1f282858BdA57

## Setting up 1 EVM.

==========================

Chain 32586980208

Estimated gas price: 3.2 gwei

Estimated total gas used for script: 11820619

Estimated amount required: 0.0378259808 ETH

==========================

###
Finding wallets for all the necessary addresses...
##
Sending transactions [0 - 4].

I'm considering improving logging throughout the repo.

Copy link
Author

linear bot commented Jan 18, 2024

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

No branches or pull requests

0 participants