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

Default 5secs fake chain wastes people time #572

Closed
DavidBruant opened this issue Feb 19, 2020 · 0 comments · Fixed by #637
Closed

Default 5secs fake chain wastes people time #572

DavidBruant opened this issue Feb 19, 2020 · 0 comments · Fixed by #637
Labels
agoric-cli package: agoric-cli

Comments

@DavidBruant
Copy link
Contributor

DavidBruant commented Feb 19, 2020

I see at least two cases of using agoric-cli:

  1. iterate (and do a modify->deploy->test-> loop) on a new smart contract
  2. test how it may feel in real life

For the first case (likely the most frequent), the fake blockchain delay feels like it only gets in the way of working
I see value for the second case, but more as an opt-in, when the first phase is over

Currently, the default value is 5secs:

const FAKE_CHAIN_DELAY =
process.env.FAKE_CHAIN_DELAY === undefined
? 5

Could it be made at least configurable? edit: my mistake, it's already configurable via an environment variable. Could it be made configurable via a command-line argument?

And as a second step reduced to 0 or something below 1sec?

@DavidBruant DavidBruant changed the title Make fake delay default to zero in agoric-cli or at least make it configurable via an cli argument Default 5secs fake chain wastes people time Feb 19, 2020
@DavidBruant DavidBruant added the agoric-cli package: agoric-cli label Feb 19, 2020
michaelfig added a commit that referenced this issue Feb 29, 2020
Closes #572

You can use `agoric start --delay=5` to get the old behaviour.

Also, changing the delay can be done at any time without
reinitialising.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
agoric-cli package: agoric-cli
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant